Bug 1589103 Don't wait for SSTabRestored event. This event doesn't get fired when back/forward causes processes switching. r=nika
authorneha <nkochar@mozilla.com>
Fri, 27 Nov 2020 20:03:09 +0000
changeset 558589 484813dea46a20436776138bc01bd320a89458e3
parent 558588 00c93ac66282823257969d14e0aa0b115fb29f14
child 558590 124e488aae3abf81f24593043ed3073dcc1645a7
push id131669
push usernlayzell@mozilla.com
push dateFri, 27 Nov 2020 20:08:46 +0000
treeherderautoland@124e488aae3a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnika
bugs1589103
milestone85.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1589103 Don't wait for SSTabRestored event. This event doesn't get fired when back/forward causes processes switching. r=nika Differential Revision: https://phabricator.services.mozilla.com/D98086
browser/base/content/test/tabs/browser_privilegedmozilla_process_pref.js
--- a/browser/base/content/test/tabs/browser_privilegedmozilla_process_pref.js
+++ b/browser/base/content/test/tabs/browser_privilegedmozilla_process_pref.js
@@ -154,35 +154,29 @@ add_task(async function process_switchin
 
     // Check that using the history back feature switches back to privileged mozilla content process.
     let promiseLocation = BrowserTestUtils.waitForLocationChange(
       gBrowser,
       TEST_HIGH1
     );
     browser.goBack();
     await promiseLocation;
-    // We will need to ensure that the process flip has fully completed so that
-    // the navigation history data will be available when we do browser.goForward();
-    await BrowserTestUtils.waitForEvent(newTab, "SSTabRestored");
     is(
       browser.frameLoader.remoteTab.osPid,
       privilegedPid,
       "Check that privileged page is still in privileged mozilla content process after history goBack."
     );
 
     // Check that using the history forward feature switches back to the web content process.
     promiseLocation = BrowserTestUtils.waitForLocationChange(
       gBrowser,
       TEST_LOW1
     );
     browser.goForward();
     await promiseLocation;
-    // We will need to ensure that the process flip has fully completed so that
-    // the navigation history data will be available when we do browser.gotoIndex(0);
-    await BrowserTestUtils.waitForEvent(newTab, "SSTabRestored");
     checkBrowserRemoteType(
       browser,
       E10SUtils.WEB_REMOTE_TYPE,
       "Check that tab runs in the web content process after using history goForward."
     );
 
     // Check that goto history index does not break the affinity.
     promiseLocation = BrowserTestUtils.waitForLocationChange(