Bug 1261842 - Tests in browser_windowStateContainer.js should wait for new browser windows to completely load. r?baku draft
authorMike Conley <mconley@mozilla.com>
Wed, 08 Jun 2016 10:48:15 -0400
changeset 392018 5d41682cd6b3e031f2bae19ec8c3c24d6dbcc861
parent 392017 8a639ae15b6f86f11c123ca39b5a08e9007bc1f0
child 392019 c13edf06c5842d7c02ef22892e4e89c0b921eb98
push id23919
push usermconley@mozilla.com
push dateFri, 22 Jul 2016 20:54:03 +0000
reviewersbaku
bugs1261842
milestone50.0a1
Bug 1261842 - Tests in browser_windowStateContainer.js should wait for new browser windows to completely load. r?baku MozReview-Commit-ID: 8SyED1dQbM5
browser/components/sessionstore/test/browser_windowStateContainer.js
--- a/browser/components/sessionstore/test/browser_windowStateContainer.js
+++ b/browser/components/sessionstore/test/browser_windowStateContainer.js
@@ -1,15 +1,14 @@
 "use strict";
 
 requestLongerTimeout(2);
 
 add_task(function* () {
-  let win = window.openDialog(location, "_blank", "chrome,all,dialog=no");
-  yield promiseWindowLoaded(win);
+  let win = yield BrowserTestUtils.openNewBrowserWindow();
 
   // Create 4 tabs with different userContextId.
   for (let userContextId = 1; userContextId < 5; userContextId++) {
     let tab = win.gBrowser.addTab("http://example.com/", {userContextId});
     yield promiseBrowserLoaded(tab.linkedBrowser);
     yield TabStateFlusher.flush(tab.linkedBrowser);
   }
 
@@ -21,18 +20,17 @@ add_task(function* () {
 
   let winState = JSON.parse(ss.getWindowState(win));
 
   for (let i = 0; i < 4; i++) {
     Assert.equal(winState.windows[0].tabs[i].userContextId, i + 1,
                  "1st Window: tabs[" + i + "].userContextId should exist.");
   }
 
-  let win2 = window.openDialog(location, "_blank", "chrome,all,dialog=no");
-  yield promiseWindowLoaded(win2);
+  let win2 = yield BrowserTestUtils.openNewBrowserWindow();
 
   // Create tabs with different userContextId, but this time we create them with
   // fewer tabs and with different order with win.
   for (let userContextId = 3; userContextId > 0; userContextId--) {
     let tab = win2.gBrowser.addTab("http://example.com/", {userContextId});
     yield promiseBrowserLoaded(tab.linkedBrowser);
     yield TabStateFlusher.flush(tab.linkedBrowser);
   }
@@ -64,35 +62,34 @@ add_task(function* () {
                  "The document has the correct userContextId");
   });
 
   yield BrowserTestUtils.closeWindow(win);
   yield BrowserTestUtils.closeWindow(win2);
 });
 
 add_task(function* () {
-  let win = window.openDialog(location, "_blank", "chrome,all,dialog=no");
-  yield promiseWindowLoaded(win);
+  let win = yield BrowserTestUtils.openNewBrowserWindow();
+  yield TabStateFlusher.flush(win.gBrowser.selectedBrowser);
 
   let tab = win.gBrowser.addTab("http://example.com/", { userContextId: 1 });
   yield promiseBrowserLoaded(tab.linkedBrowser);
   yield TabStateFlusher.flush(tab.linkedBrowser);
 
   // win should have 1 default tab, and 1 container tab.
   Assert.equal(win.gBrowser.tabs.length, 2, "win should have 2 tabs");
 
   let winState = JSON.parse(ss.getWindowState(win));
 
   for (let i = 0; i < 2; i++) {
     Assert.equal(winState.windows[0].tabs[i].userContextId, i,
                  "1st Window: tabs[" + i + "].userContextId should be " + i);
   }
 
-  let win2 = window.openDialog(location, "_blank", "chrome,all,dialog=no");
-  yield promiseWindowLoaded(win2);
+  let win2 = yield BrowserTestUtils.openNewBrowserWindow();
 
   let tab2 = win2.gBrowser.addTab("http://example.com/", { userContextId : 1 });
   yield promiseBrowserLoaded(tab2.linkedBrowser);
   yield TabStateFlusher.flush(tab2.linkedBrowser);
 
   // Move the first normal tab to end, so the first tab of win2 will be a
   // container tab.
   win2.gBrowser.moveTabTo(win2.gBrowser.tabs[0], win2.gBrowser.tabs.length - 1);