1093404 - e10s - fix browser_bug462673.js

Status: RESOLVED FIXED

(Firefox :: Tabbed Browser, defect)

Description

[:Gijs (he/him)]

The test seems to depend on the content window opening another tab, which happens asynchronously, and which makes the test fail:

988 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug462673.js | test_bug462673.html has opened a second tab - Got 1, expected 2
Stack trace:
chrome://mochikit/content/browser-test.js:test_is:828
chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug462673.js:runs<:3
chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug462673.js:runOneTest/</</<:54
chrome://mochikit/content/browser-test.js:testScope/test_executeSoon/<.run:858
null:null:0
989 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug462673.js | dependent tab is selected - Got [object XULElement], expected null
Stack trace:
chrome://mochikit/content/browser-test.js:test_is:828
chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug462673.js:runs<:4
chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug462673.js:runOneTest/</</<:54
chrome://mochikit/content/browser-test.js:testScope/test_executeSoon/<.run:858
null:null:0

Comment 1

[Neil Deakin]

Attachment: Fix test

Comment 2

[:Gijs (he/him)]

Comment on attachment 8576758 [details] [diff] [review]
Fix test

Review of attachment 8576758 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/base/content/test/general/browser_bug462673.js
@@ +9,5 @@
> +  is(win.gBrowser.selectedTab, tab.nextSibling, "dependent tab is selected");
> +  win.gBrowser.removeTab(tab);
> +
> +  // Closing a tab will also close its parent chrome window, but async
> +  yield promiseWaitForCondition(() => win.closed);

use yield promiseWindowWillBeClosed(win); instead to avoid the timeout-based loop here.

@@ +35,2 @@
>    win.close();
> +  yield promiseWaitForCondition(() => win.closed);

Instead of these two lines, use yield promiseWindowClosed(win), which calls win.close() and doesn't race if we ever start closing it sync from win.close...

Comment 3

[:Gijs (he/him)]

(if you want to move both of these to BrowserTestUtils, that would be welcome!)

Comment 4

[Neil Deakin]

https://hg.mozilla.org/integration/mozilla-inbound/rev/328d04f6980a

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/328d04f6980a