Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Gecko Bug 1557548] part1 : remove 'controls' attribute from video. #17321

Merged
merged 1 commit into from Jun 14, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

It's no need to add 'controls' attribute in this test because it's totally unrelated with navigation and we even have to add extra offset for the cue text in the reference file.

Differential Revision: https://phabricator.services.mozilla.com/D34777

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1557548
gecko-commit: cd473827dfd4ecf8d33f26d1764e5d51ebf2c393
gecko-integration-branch: autoland
gecko-reviewers: heycam

It's no need to add 'controls' attribute in this test because it's totally unrelated with navigation and we even have to add extra offset for the cue text in the reference file.

Differential Revision: https://phabricator.services.mozilla.com/D34777

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1557548
gecko-commit: cd473827dfd4ecf8d33f26d1764e5d51ebf2c393
gecko-integration-branch: autoland
gecko-reviewers: heycam
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already reviewed downstream.

@moz-wptsync-bot moz-wptsync-bot merged commit 93eb4c0 into master Jun 14, 2019
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1557548 branch June 14, 2019 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants