Bug 1576407: Part 5 - Update test expectations to handler cross-process window.opener being available. r=farre
☠☠ backed out by be74f006fdfd ☠ ☠
authorKris Maglione <maglione.k@gmail.com>
Wed, 28 Aug 2019 18:07:41 +0000
changeset 554236 38e4ea4314452422fe5fcfb8b1b557201ae9c0f8
parent 554235 19bf40067233299dbe4870e3a5698ae67027dcd9
child 554237 465c399713381aae67efede22e3496602d2139b5
push id2165
push userffxbld-merge
push dateMon, 14 Oct 2019 16:30:58 +0000
treeherdermozilla-release@0eae18af659f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfarre
bugs1576407
milestone70.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 1576407: Part 5 - Update test expectations to handler cross-process window.opener being available. r=farre This also sets the fission.rebuild_frameloaders_on_remoteness_change=true preference for some mochitest directories which require it for cross-process window.opener to work in top-level windows, and makes a minor change to the hack in browser_temporary_permissions.js to keep it passing reliably in try runs. Differential Revision: https://phabricator.services.mozilla.com/D43694
browser/base/content/test/permissions/browser_temporary_permissions.js
dom/tests/mochitest/bugs/mochitest.ini
dom/tests/mochitest/localstorage/mochitest.ini
dom/tests/mochitest/sessionstorage/mochitest.ini
dom/tests/mochitest/whatwg/mochitest.ini
toolkit/components/extensions/test/mochitest/mochitest-common.ini
--- a/browser/base/content/test/permissions/browser_temporary_permissions.js
+++ b/browser/base/content/test/permissions/browser_temporary_permissions.js
@@ -67,17 +67,17 @@ add_task(async function testTempPermissi
       "popupshown"
     );
 
     // Request a permission.
     await ContentTask.spawn(browser, uri.host, async function(host0) {
       // FIXME(Fission): The load event fires before cross-origin iframes have
       // loaded (bug 1559841).
       if (content.SpecialPowers.useRemoteSubframes) {
-        for (let i = 0; i < 200; i++) {
+        for (let i = 0; i < 800; i++) {
           await new Promise(resolve => content.setTimeout(resolve, 0));
         }
       }
 
       let frame = content.document.getElementById("frame");
 
       await content.SpecialPowers.spawn(frame, [host0], async function(host) {
         const { E10SUtils } = ChromeUtils.import(
--- a/dom/tests/mochitest/bugs/mochitest.ini
+++ b/dom/tests/mochitest/bugs/mochitest.ini
@@ -33,16 +33,18 @@ support-files =
   file_redirector.sjs
   file_prime_cookie.html
   test1_bug369306.html
   test2_bug369306.html
   test1_bug622361.html
   test2_bug622361.html
   file1_bug414291.html
   file2_bug414291.html
+prefs =
+  fission.rebuild_frameloaders_on_remoteness_change=true
 
 [test_DOMWindowCreated_chromeonly.html]
 [test_bug132255.html]
 [test_bug1551425_reference_to_childnodes_nodelist_keeps_childnodes_alive.html]
 [test_bug159849.html]
 [test_bug289714.html]
 [test_bug260264.html]
 skip-if = fission || toolkit == 'android'
--- a/dom/tests/mochitest/localstorage/mochitest.ini
+++ b/dom/tests/mochitest/localstorage/mochitest.ini
@@ -13,16 +13,18 @@ support-files =
   frameSlaveNotEqual.html
   interOriginFrame.js
   interOriginTest.js
   interOriginTest2.js
   localStorageCommon.js
   frameLocalStorageSessionOnly.html
   file_tryAccessSessionStorage.html
   windowProxy.html
+prefs =
+  fission.rebuild_frameloaders_on_remoteness_change=true
 
 [test_brokenUTF-16.html]
 [test_bug600307-DBOps.html]
 [test_bug746272-1.html]
 [test_bug746272-2.html]
 skip-if = os == "android" || verify # bug 962029
 [test_cookieBlock.html]
 [test_embededNulls.html]
@@ -48,11 +50,10 @@ skip-if = toolkit == 'android' #TIMED_OU
 fail-if = fission
 [test_localStorageQuota.html]
 skip-if = toolkit == 'android' #TIMED_OUT
 [test_localStorageQuotaSessionOnly.html]
 skip-if = toolkit == 'android' || (verify && (os == 'linux' || os == 'mac' || os == 'win')) #TIMED_OUT
 [test_localStorageQuotaSessionOnly2.html]
 skip-if = true # bug 1347690
 [test_localStorageReplace.html]
-fail-if = fission
 skip-if = toolkit == 'android'
 [test_storageConstructor.html]
--- a/dom/tests/mochitest/sessionstorage/mochitest.ini
+++ b/dom/tests/mochitest/sessionstorage/mochitest.ini
@@ -2,19 +2,20 @@
 support-files =
   file_http.html
   file_https.html
   frameEqual.html
   frameNotEqual.html
   frameReplace.html
   interOriginSlave.js
   interOriginTest.js
+prefs =
+  fission.rebuild_frameloaders_on_remoteness_change=true
 
 [test_sessionStorageBase.html]
 [test_sessionStorageBaseSessionOnly.html]
 [test_sessionStorageClone.html]
-fail-if = fission
 skip-if = toolkit == 'android'
 [test_sessionStorageHttpHttps.html]
 skip-if = toolkit == 'android' #TIMED_OUT
 [test_sessionStorageReplace.html]
 fail-if = fission
 [test_sessionStorageUsage.html]
--- a/dom/tests/mochitest/whatwg/mochitest.ini
+++ b/dom/tests/mochitest/whatwg/mochitest.ini
@@ -10,35 +10,34 @@ support-files =
   postMessage_onOther.html
   postMessage_origin_helper.xhtml
   postMessage_override_helper.html
   postMessage_structured_clone_helper.html
   postMessage_structured_clone_helper.js
   postMessage_throw_helper.html
   postMessage_transfer_helper.html
   postMessage_userpass_helper.html
+prefs =
+  fission.rebuild_frameloaders_on_remoteness_change=true
 
 [test_bug477323.html]
 [test_document_scripts.html]
 [test_MessageEvent_dispatchToOther.html]
 [test_MessageEvent.html]
 [test_postMessage_basehref.html]
 [test_postMessage_closed.html]
 skip-if = toolkit == 'android' #bug 894914 - wrong data - got FAIL, expected message
 [test_postMessage_hash.html]
 [test_postMessage.html]
 [test_postMessage_idn.xhtml]
 [test_postMessage_joined.html]
 [test_postMessage_onOther.html]
-skip-if = fission #Bug 1571273
 [test_postMessage_origin.xhtml]
 [test_postMessage_override.html]
 skip-if = fission
 [test_postMessage_special.xhtml]
 [test_postMessage_structured_clone.html]
-skip-if = fission #Bug 1570918
 [test_postMessage_throw.html]
 [test_postMessage_transfer.html]
-skip-if = fission #Bug 1571208
 [test_postMessage_userpass.html]
 [test_bug500328.html]
 skip-if = true || toolkit=='android' # bug 696306, #TIMED_OUT android
 support-files = file_bug500328_1.html file_bug500328_2.html
--- a/toolkit/components/extensions/test/mochitest/mochitest-common.ini
+++ b/toolkit/components/extensions/test/mochitest/mochitest-common.ini
@@ -80,17 +80,16 @@ skip-if =
   (verify && debug && (os == 'linux')) ||
   fission # Fails intermittently under Fission.
 [test_ext_contentscript_devtools_metadata.html]
 skip-if = (!debug && android_version == '18') #Bug 1523193
 [test_ext_contentscript_incognito.html]
 skip-if = os == 'android' # Android does not support multiple windows.
 [test_ext_contentscript_permission.html]
 [test_ext_cookies.html]
-fail-if = fission
 [test_ext_cookies_containers.html]
 [test_ext_cookies_expiry.html]
 [test_ext_cookies_first_party.html]
 [test_ext_cookies_incognito.html]
 skip-if = os == 'android' # Bug 1513544 Android does not support multiple windows.
 [test_ext_cookies_permissions_bad.html]
 [test_ext_cookies_permissions_good.html]
 [test_ext_exclude_include_globs.html]