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 1558646] Fail test by assert on, and log, unexpected events in WPT MS-ME-firstframe. #17312

Merged
merged 2 commits into from Jun 13, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1558646
gecko-commit: a859291efa4e79246fcdb60d5b20b088fe993d7e
gecko-integration-branch: autoland
gecko-reviewers: jib

…frame.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1558646
gecko-commit: a859291efa4e79246fcdb60d5b20b088fe993d7e
gecko-integration-branch: autoland
gecko-reviewers: jib
Stopping the tracks as happens in cleanup makes the media element end playback,
which fires durationchange. It doesn't fail the test since it already finished,
but having it show up in the log is distracting.

Depends on D34707

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1558646
gecko-commit: d5b7f97785fa12ef61743d4d68ccfb9d00071aab
gecko-integration-branch: autoland
gecko-reviewers: jib
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 222e30c into master Jun 13, 2019
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1558646 branch June 13, 2019 22:13
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