diff --git a/docshell/base/crashtests/crashtests.list b/docshell/base/crashtests/crashtests.list index 1fee6f2a360d..29e5358a65ad 100644 --- a/docshell/base/crashtests/crashtests.list +++ b/docshell/base/crashtests/crashtests.list @@ -11,7 +11,7 @@ load 500328-1.html load 514779-1.xhtml load 614499-1.html load 678872-1.html -skip-if(Android) pref(dom.disable_open_during_load,false) load 914521.html # Android bug 1584562 +skip pref(dom.disable_open_during_load,false) load 914521.html # Android bug 1584562, process selection bug 1599662. pref(browser.send_pings,true) asserts(0-2) load 1257730-1.html # bug 566159 load 1331295.html load 1341657.html diff --git a/dom/ipc/BrowserParent.cpp b/dom/ipc/BrowserParent.cpp index 08bd2dd84d9d..92da4d167dc7 100644 --- a/dom/ipc/BrowserParent.cpp +++ b/dom/ipc/BrowserParent.cpp @@ -903,14 +903,6 @@ void BrowserParent::InitRendering() { layers::LayersId layersId = mRemoteLayerTreeOwner.GetLayersId(); AddBrowserParentToTable(layersId, this); - RefPtr frameLoader = GetFrameLoader(); - if (frameLoader) { - nsIFrame* frame = frameLoader->GetPrimaryFrameOfOwningContent(); - if (frame) { - frame->InvalidateFrame(); - } - } - TextureFactoryIdentifier textureFactoryIdentifier; mRemoteLayerTreeOwner.GetTextureFactoryIdentifier(&textureFactoryIdentifier); Unused << SendInitRendering(textureFactoryIdentifier, layersId, diff --git a/layout/tools/reftest/reftest-content.js b/layout/tools/reftest/reftest-content.js index 259518df9d27..d36fea5763ad 100644 --- a/layout/tools/reftest/reftest-content.js +++ b/layout/tools/reftest/reftest-content.js @@ -110,9 +110,6 @@ function OnInitialLoad() removeEventListener("load", OnInitialLoad, true); gDebug = Cc[DEBUG_CONTRACTID].getService(Ci.nsIDebug2); - if (gDebug.isDebugBuild) { - gAssertionCount = gDebug.assertionCount; - } var env = Cc[ENVIRONMENT_CONTRACTID].getService(Ci.nsIEnvironment); gVerbose = !!env.get("MOZ_REFTEST_VERBOSE"); diff --git a/layout/tools/reftest/reftest.jsm b/layout/tools/reftest/reftest.jsm index bd2bfbefbc49..9142d2d018f8 100644 --- a/layout/tools/reftest/reftest.jsm +++ b/layout/tools/reftest/reftest.jsm @@ -18,12 +18,9 @@ Cu.import("resource://reftest/manifest.jsm", this); Cu.import("resource://reftest/StructuredLog.jsm", this); Cu.import("resource://reftest/PerTestCoverageUtils.jsm", this); Cu.import("resource://gre/modules/Services.jsm"); +Cu.import("resource://gre/modules/NetUtil.jsm"); Cu.import('resource://gre/modules/XPCOMUtils.jsm'); -const { E10SUtils } = ChromeUtils.import( - "resource://gre/modules/E10SUtils.jsm" -); - XPCOMUtils.defineLazyGetter(this, "OS", function() { const { OS } = Cu.import("resource://gre/modules/osfile.jsm"); return OS; @@ -227,7 +224,7 @@ function OnRefTestLoad(win) g.browserMessageManager = g.browser.frameLoader.messageManager; // The content script waits for the initial onload, then notifies // us. - RegisterMessageListenersAndLoadContentScript(false); + RegisterMessageListenersAndLoadContentScript(); } function InitAndStartRefTests() @@ -672,41 +669,7 @@ function StartCurrentTest() } } -// A simplified version of the function with the same name in tabbrowser.js. -function updateBrowserRemotenessByURL(aBrowser, aURL) { - let remoteType = E10SUtils.getRemoteTypeForURI( - aURL, - aBrowser.ownerGlobal.docShell.nsILoadContext.useRemoteTabs, - aBrowser.ownerGlobal.docShell.nsILoadContext.useRemoteSubframes, - aBrowser.remoteType, - aBrowser.currentURI - ); - // Things get confused if we switch to not-remote - // for chrome:// URIs, so lets not for now. - if (remoteType == E10SUtils.NOT_REMOTE && - g.browserIsRemote) { - remoteType = aBrowser.remoteType; - } - if (aBrowser.remoteType != remoteType) { - if (remoteType == E10SUtils.NOT_REMOTE) { - aBrowser.removeAttribute("remote"); - aBrowser.removeAttribute("remoteType"); - } else { - aBrowser.setAttribute("remote", "true"); - aBrowser.setAttribute("remoteType", remoteType); - } - aBrowser.changeRemoteness({ remoteType }); - aBrowser.construct(); - - g.browserMessageManager = aBrowser.frameLoader.messageManager; - RegisterMessageListenersAndLoadContentScript(true); - return new Promise(resolve => { g.resolveContentReady = resolve; }); - } - - return Promise.resolve(); -} - -async function StartCurrentURI(aURLTargetType) +function StartCurrentURI(aURLTargetType) { const isStartingRef = (aURLTargetType == URL_TARGET_TYPE_REFERENCE); @@ -804,8 +767,6 @@ async function StartCurrentURI(aURLTargetType) gDumpFn("REFTEST TEST-LOAD | " + g.currentURL + " | " + currentTest + " / " + g.totalTests + " (" + Math.floor(100 * (currentTest / g.totalTests)) + "%)\n"); TestBuffer("START " + g.currentURL); - await updateBrowserRemotenessByURL(g.browser, g.currentURL); - var type = g.urls[0].type if (TYPE_SCRIPT == type) { SendLoadScriptTest(g.currentURL, g.loadTimeout); @@ -1478,7 +1439,7 @@ function RestoreChangedPreferences() } } -function RegisterMessageListenersAndLoadContentScript(aReload) +function RegisterMessageListenersAndLoadContentScript() { g.browserMessageManager.addMessageListener( "reftest:AssertionCount", @@ -1551,10 +1512,6 @@ function RegisterMessageListenersAndLoadContentScript(aReload) g.browserMessageManager.loadFrameScript("resource://reftest/reftest-content.js", true, true); - if (aReload) { - return; - } - ChromeUtils.registerWindowActor("ReftestFission", { parent: { moduleURI: "resource://reftest/ReftestFissionParent.jsm", @@ -1574,19 +1531,8 @@ function RecvAssertionCount(count) function RecvContentReady(info) { - if (g.resolveContentReady) { - // Focus the content browser. - if (g.focusFilterMode != FOCUS_FILTER_NON_NEEDS_FOCUS_TESTS) { - if (Services.focus.activeWindow != g.containingWindow) { - Focus(); - } - } - g.resolveContentReady(); - g.resolveContentReady = null; - } else { - g.contentGfxInfo = info.gfx; - InitAndStartRefTests(); - } + g.contentGfxInfo = info.gfx; + InitAndStartRefTests(); return { remote: g.browserIsRemote }; }