diff --git a/packages/browsers/src/browser-data/firefox.ts b/packages/browsers/src/browser-data/firefox.ts index 33033cc8fe326..7c816c4284801 100644 --- a/packages/browsers/src/browser-data/firefox.ts +++ b/packages/browsers/src/browser-data/firefox.ts @@ -220,12 +220,6 @@ function defaultProfilePreferences( // Make sure opening about:addons will not hit the network 'extensions.webservice.discoverURL': `http://${server}/dummy/discoveryURL`, - // Temporarily force disable BFCache in parent (https://bit.ly/bug-1732263) - 'fission.bfcacheInParent': false, - - // Force all web content to use a single content process - 'fission.webContentIsolationStrategy': 0, - // Allow the application to have focus even it runs in the background 'focusmanager.testmode': true, // Disable useragent updates diff --git a/packages/puppeteer-core/src/node/FirefoxLauncher.ts b/packages/puppeteer-core/src/node/FirefoxLauncher.ts index e7ebe9f43b749..cd610d2451648 100644 --- a/packages/puppeteer-core/src/node/FirefoxLauncher.ts +++ b/packages/puppeteer-core/src/node/FirefoxLauncher.ts @@ -113,7 +113,21 @@ export class FirefoxLauncher extends ProductLauncher { await createProfile(SupportedBrowsers.FIREFOX, { path: userDataDir, - preferences: extraPrefsFirefox, + preferences: { + ...extraPrefsFirefox, + ...(options.protocol === 'cdp' + ? { + // Temporarily force disable BFCache in parent (https://bit.ly/bug-1732263) + 'fission.bfcacheInParent': false, + } + : {}), + // Force all web content to use a single content process. TODO: remove + // this once Firefox supports mouse event dispatch from the main frame + // context. Once this happens, webContentIsolationStrategy should only + // be set for CDP. See + // https://bugzilla.mozilla.org/show_bug.cgi?id=1773393 + 'fission.webContentIsolationStrategy': 0, + }, }); let firefoxExecutable: string; diff --git a/test/TestExpectations.json b/test/TestExpectations.json index 0106f550496c4..b56c35c90d3f3 100644 --- a/test/TestExpectations.json +++ b/test/TestExpectations.json @@ -2765,12 +2765,24 @@ "parameters": ["cdp", "firefox"], "expectations": ["FAIL"] }, + { + "testIdPattern": "[oopif.spec] OOPIF clickablePoint, boundingBox, boxModel should work for elements inside OOPIFs", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["firefox", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[oopif.spec] OOPIF should keep track of a frames OOP state", "platforms": ["darwin", "linux", "win32"], "parameters": ["firefox", "webDriverBiDi"], "expectations": ["PASS"] }, + { + "testIdPattern": "[oopif.spec] OOPIF should provide access to elements", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["firefox", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[oopif.spec] OOPIF should support lazy OOP frames", "platforms": ["darwin", "linux", "win32"],