Bug 1336763 - Don't wait for response from content process in browser_closeTab.js UITour test to avoid a timeout. r?MattN
The closeTab API call from UITour tells the parent process to close the current tab,
and doing so might end up disconnecting the message manager. This means that the
Promise that the ContentTask returns may never resolve. Instead of waiting for it
to resolve, we just wait for the TabClose event in the parent process.
MozReview-Commit-ID: Ci7ck9j4llK
--- a/browser/components/uitour/test/browser_closeTab.js
+++ b/browser/components/uitour/test/browser_closeTab.js
@@ -7,12 +7,17 @@ var gContentAPI;
var gContentWindow;
add_task(setup_UITourTest);
add_UITour_task(function* test_closeTab() {
// Setting gTestTab to null indicates that the tab has already been closed,
// and if this does not happen the test run will fail.
let closePromise = BrowserTestUtils.waitForEvent(gBrowser.tabContainer, "TabClose");
- yield gContentAPI.closeTab();
+ // In the e10s-case, having content request a tab close might mean
+ // that the ContentTask used to send this closeTab message won't
+ // get a response (since the message manager may have closed down).
+ // So we ignore the Promise that closeTab returns, and use the TabClose
+ // event to tell us when the tab has gone away.
+ gContentAPI.closeTab();
yield closePromise;
gTestTab = null;
});