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 1477977] [marionette] Handle "tabmodal-dialog-loaded" to observe new tab modal dialogs. #17319

Merged
merged 2 commits into from Jun 19, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1477977
gecko-commit: e9e20d087e338275a4ceae79e91d7a8e2f882f91
gecko-integration-branch: autoland
gecko-reviewers: maja_zf

… dialogs.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1477977
gecko-commit: e9e20d087e338275a4ceae79e91d7a8e2f882f91
gecko-integration-branch: autoland
gecko-reviewers: maja_zf
…tus between tabs.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1477977
gecko-commit: 4651b31ffe5c012b5112fc3047c29d0d4a8f9594
gecko-integration-branch: autoland
gecko-reviewers: webdriver-reviewers, automatedtester
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.

@whimboo
Copy link
Contributor

whimboo commented Jun 13, 2019

The failures here is Chrome mostly failing with a timeout failure for the new tab-modal test, which checks for retained tab modals across tabs. Looks like it cannot handle it yet. CC'ing @JohnChen0 for info.

Update: Whereby this is only for the verify job, with Chrome experimental it is all fine. So maybe there was a race and the clean-up of a test was not successful.

@jgraham jgraham merged commit 15801d1 into master Jun 19, 2019
@whimboo whimboo deleted the gecko/1477977 branch June 19, 2019 12:54
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

4 participants