Closed
Bug 1502509
Opened 6 years ago
Closed 6 years ago
browser.xhtml: tests hang that use waitForNewWindow
Categories
(Testing :: Mochitest, defect)
Tracking
(firefox65 fixed)
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: bdahl, Assigned: bdahl)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
In browser.xhtml browser chrome tests that use waitForNewWindow hang. This is because with browser.xhtml the focus/activate events can be triggered before the DOMContentLoaded event.
Failing tests:
browser/base/content/test/contextMenu/browser_contextmenu_linkopen.js
browser/base/content/test/popups/browser_popupUI.js
browser/components/sessionstore/test/browser_394759_behavior.js
browser/components/extensions/test/browser/browser_ext_windows.js
browser/base/content/test/general/browser_newWindowDrop.js
browser/base/content/test/general/browser_new_http_window_opened_from_file_tab.js
browser/components/privatebrowsing/test/browser/browser_privatebrowsing_newtab_from_popup.js
browser/components/extensions/test/browser/test-oop-extensions/browser_ext_windows.js
dom/tests/browser/browser_noopener.js
toolkit/components/windowwatcher/test/browser_new_content_window_from_chrome_principal.js
Assignee | ||
Comment 1•6 years ago
|
||
In browser.xhtml the focus and activate events can fire before the
DOMContentLoaded event.
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/1e422e5034f1
Handle different event order when waiting for window to open. r=Gijs
Comment 3•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•