Bug 1727323 - [devtools] Enable bfcache in parent in tests that have been fixed thanks to server targets. r=bomsy

All these tests most likely have been fixed with the enabling of server targets. Because bfcache in parent was introducing more cross process navigations, being initiated from the server, having the support of server targets was uterly important to support bfcacheInParent. Differential Revision: https://phabricator.services.mozilla.com/D123448
2024-11-24 21:31:04 +00:00 · 2021-08-25 17:16:27 +00:00 · 2021-08-25 17:16:27 +00:00 · 3b693fede5
commit 3b693fede5
parent f225d7e930
42 changed files with 0 additions and 258 deletions
--- a/devtools/client/application/test/browser/browser_application_panel_list-domain-workers.js
+++ b/devtools/client/application/test/browser/browser_application_panel_list-domain-workers.js
@ -16,12 +16,6 @@ const EMPTY_URL = (URL_ROOT + "resources/service-workers/empty.html").replace(
 );

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  await enableApplicationPanel();

  const { panel, commands, tab } = await openNewTabAndApplicationPanel(
--- a/devtools/client/application/test/browser/browser_application_panel_list-several-workers.js
+++ b/devtools/client/application/test/browser/browser_application_panel_list-several-workers.js
@ -12,12 +12,6 @@ const SIMPLE_URL = URL_ROOT + "resources/service-workers/simple.html";
 const OTHER_SCOPE_URL = URL_ROOT + "resources/service-workers/scope-page.html";

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  await enableApplicationPanel();

  const { panel, commands, tab } = await openNewTabAndApplicationPanel(
--- a/devtools/client/debugger/test/mochitest/browser_dbg-link-reload.js
+++ b/devtools/client/debugger/test/mochitest/browser_dbg-link-reload.js
@ -7,12 +7,6 @@
 * It seems to cause different codepath compared to F5.
 */
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const dbg = await initDebugger("doc-reload-link.html", "doc-reload-link.html");
  const {
    selectors: { getSelectedSource, getIsPaused, getCurrentThread },
--- a/devtools/client/debugger/test/mochitest/browser_dbg-preview-module.js
+++ b/devtools/client/debugger/test/mochitest/browser_dbg-preview-module.js
@ -5,12 +5,6 @@
 // Test hovering in a script that is paused on load
 // and doesn't have functions.
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const dbg = await initDebugger("doc-scripts.html");

  const onNavigated = navigate(dbg, "doc-on-load.html");
--- a/devtools/client/debugger/test/mochitest/browser_dbg-watchpoints.js
+++ b/devtools/client/debugger/test/mochitest/browser_dbg-watchpoints.js
@ -8,12 +8,6 @@
 // then removing and adding a watchpoint during the same pause

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const dbg = await initDebugger("doc-sources.html");

  // Do not await for navigation as an early breakpoint pauses the document load
--- a/devtools/client/framework/test/browser_toolbox_fission_navigation.js
+++ b/devtools/client/framework/test/browser_toolbox_fission_navigation.js
@ -8,12 +8,6 @@ const EXAMPLE_NET_URI =
  "http://example.net/document-builder.sjs?html=<div id=net>net";

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  info("Test with server side target switching ENABLED");
  await pushPref("devtools.target-switching.server.enabled", true);
  await testCase();
--- a/devtools/client/inspector/grids/test/browser_grids_restored-after-reload.js
+++ b/devtools/client/inspector/grids/test/browser_grids_restored-after-reload.js
@ -34,12 +34,6 @@ const OTHER_URI = `
 ignoreGetGridsPromiseRejections();

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  await addTab("data:text/html;charset=utf-8," + encodeURIComponent(TEST_URI));
  const { gridInspector, inspector } = await openLayoutView();
  const { document: doc } = gridInspector;
--- a/devtools/client/inspector/grids/test/browser_grids_restored-multiple-grids-after-reload.js
+++ b/devtools/client/inspector/grids/test/browser_grids_restored-multiple-grids-after-reload.js
@ -33,12 +33,6 @@ const TEST_URI = `
 ignoreGetGridsPromiseRejections();

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  await pushPref("devtools.gridinspector.maxHighlighters", 3);
  await addTab("data:text/html;charset=utf-8," + encodeURIComponent(TEST_URI));
  const { gridInspector, inspector } = await openLayoutView();
--- a/devtools/client/inspector/test/browser_inspector_highlighter-05.js
+++ b/devtools/client/inspector/test/browser_inspector_highlighter-05.js
@ -49,12 +49,6 @@ const TEST_URL =
 `);

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  info("Open the inspector to a blank page.");
  const { inspector } = await openInspectorForURL("about:blank");

--- a/devtools/client/inspector/test/browser_inspector_highlighter-eyedropper-xul.js
+++ b/devtools/client/inspector/test/browser_inspector_highlighter-eyedropper-xul.js
@ -11,12 +11,6 @@ const TEST_URL_2 =
  "data:text/html;charset=utf-8,<h1 style='color:red'>HTML test page</h1>";

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { inspector } = await openInspectorForURL(TEST_URL);

  info("Check the inspector toolbar");
--- a/devtools/client/inspector/test/browser_inspector_startup.js
+++ b/devtools/client/inspector/test/browser_inspector_startup.js
@ -34,12 +34,6 @@ const TEST_URL =
  "</html>";

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { inspector, tab } = await openInspectorForURL("about:blank");

  const domContentLoaded = waitForLinkedBrowserEvent(tab, "DOMContentLoaded");
--- a/devtools/client/memory/test/browser/browser_memory_fission_switch_target.js
+++ b/devtools/client/memory/test/browser/browser_memory_fission_switch_target.js
@ -13,12 +13,6 @@ const EXPECTED_ELEMENT_IN_PARENT_PROCESS = "button";
 const EXPECTED_ELEMENT_IN_CONTENT_PROCESS = "section";

 add_task(async () => {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  info("Open the memory panel with empty page");
  const tab = await addTab();
  const { panel } = await openMemoryPanel(tab);
--- a/devtools/client/netmonitor/test/browser_net_block-context.js
+++ b/devtools/client/netmonitor/test/browser_net_block-context.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { tab, monitor } = await initNetMonitor(SIMPLE_URL, {
    requestCount: 1,
  });
--- a/devtools/client/netmonitor/test/browser_net_block-draganddrop.js
+++ b/devtools/client/netmonitor/test/browser_net_block-draganddrop.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  class DataTransfer {
    constructor() {
      this.BLOCKING_URL =
--- a/devtools/client/netmonitor/test/browser_net_block.js
+++ b/devtools/client/netmonitor/test/browser_net_block.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { tab, monitor } = await initNetMonitor(SIMPLE_URL, {
    requestCount: 1,
  });
--- a/devtools/client/netmonitor/test/browser_net_cause_source_map.js
+++ b/devtools/client/netmonitor/test/browser_net_cause_source_map.js
@ -13,12 +13,6 @@ const CAUSE_URL = EXAMPLE_URL + CAUSE_FILE_NAME;
 const N_EXPECTED_REQUESTS = 4;

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  // the initNetMonitor function clears the network request list after the
  // page is loaded. That's why we first load a bogus page from SIMPLE_URL,
  // and only then load the real thing from CAUSE_URL - we want to catch
--- a/devtools/client/netmonitor/test/browser_net_charts-01.js
+++ b/devtools/client/netmonitor/test/browser_net_charts-01.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });

  info("Starting test... ");
--- a/devtools/client/netmonitor/test/browser_net_charts-02.js
+++ b/devtools/client/netmonitor/test/browser_net_charts-02.js
@ -9,12 +9,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { L10N } = require("devtools/client/netmonitor/src/utils/l10n");

  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/netmonitor/test/browser_net_charts-03.js
+++ b/devtools/client/netmonitor/test/browser_net_charts-03.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { L10N } = require("devtools/client/netmonitor/src/utils/l10n");

  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/netmonitor/test/browser_net_charts-04.js
+++ b/devtools/client/netmonitor/test/browser_net_charts-04.js
@ -9,12 +9,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { L10N } = require("devtools/client/netmonitor/src/utils/l10n");

  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/netmonitor/test/browser_net_charts-05.js
+++ b/devtools/client/netmonitor/test/browser_net_charts-05.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { L10N } = require("devtools/client/netmonitor/src/utils/l10n");

  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/netmonitor/test/browser_net_charts-06.js
+++ b/devtools/client/netmonitor/test/browser_net_charts-06.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { L10N } = require("devtools/client/netmonitor/src/utils/l10n");

  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/netmonitor/test/browser_net_charts-07.js
+++ b/devtools/client/netmonitor/test/browser_net_charts-07.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { L10N } = require("devtools/client/netmonitor/src/utils/l10n");

  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/netmonitor/test/browser_net_columns_showhide.js
+++ b/devtools/client/netmonitor/test/browser_net_columns_showhide.js
@ -7,12 +7,6 @@
 * Test showing/hiding columns.
 */
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
  info("Starting test... ");

--- a/devtools/client/netmonitor/test/browser_net_edit_resend_cancel.js
+++ b/devtools/client/netmonitor/test/browser_net_edit_resend_cancel.js
@ -8,12 +8,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
  info("Starting test... ");

--- a/devtools/client/netmonitor/test/browser_net_edit_resend_caret.js
+++ b/devtools/client/netmonitor/test/browser_net_edit_resend_caret.js
@ -10,12 +10,6 @@
 */

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
  info("Starting test... ");

--- a/devtools/client/netmonitor/test/browser_net_initiator.js
+++ b/devtools/client/netmonitor/test/browser_net_initiator.js
@ -133,12 +133,6 @@ const EXPECTED_REQUESTS = [
 ];

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  // the initNetMonitor function clears the network request list after the
  // page is loaded. That's why we first load a bogus page from SIMPLE_URL,
  // and only then load the real thing from INITIATOR_URL - we want to catch
--- a/devtools/client/netmonitor/test/browser_net_telemetry_edit_resend.js
+++ b/devtools/client/netmonitor/test/browser_net_telemetry_edit_resend.js
@ -9,12 +9,6 @@ const ALL_CHANNELS = Ci.nsITelemetry.DATASET_ALL_CHANNELS;
 * Test the edit_resend telemetry event.
 */
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
  info("Starting test... ");

--- a/devtools/client/netmonitor/test/browser_net_telemetry_filters_changed.js
+++ b/devtools/client/netmonitor/test/browser_net_telemetry_filters_changed.js
@ -9,12 +9,6 @@ const ALL_CHANNELS = Ci.nsITelemetry.DATASET_ALL_CHANNELS;
 * Test the filters_changed telemetry event.
 */
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
  info("Starting test... ");

--- a/devtools/client/netmonitor/test/browser_net_telemetry_sidepanel_changed.js
+++ b/devtools/client/netmonitor/test/browser_net_telemetry_sidepanel_changed.js
@ -9,12 +9,6 @@ const ALL_CHANNELS = Ci.nsITelemetry.DATASET_ALL_CHANNELS;
 * Test the sidepanel_changed telemetry event.
 */
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
  info("Starting test... ");

--- a/devtools/client/netmonitor/test/browser_net_websocket_stacks.js
+++ b/devtools/client/netmonitor/test/browser_net_websocket_stacks.js
@ -48,12 +48,6 @@ const EXPECTED_REQUESTS = {
 };

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  // Load a different URL first to instantiate the network monitor before we
  // load the page we're really interested in.
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/netmonitor/test/browser_net_worker_stacks.js
+++ b/devtools/client/netmonitor/test/browser_net_worker_stacks.js
@ -83,12 +83,6 @@ const EXPECTED_REQUESTS = [
 ];

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  // Load a different URL first to instantiate the network monitor before we
  // load the page we're really interested in.
  const { monitor } = await initNetMonitor(SIMPLE_URL, { requestCount: 1 });
--- a/devtools/client/responsive/test/browser/browser_navigation.js
+++ b/devtools/client/responsive/test/browser/browser_navigation.js
@ -13,12 +13,6 @@ const DUMMY_3_URL = "http://example.com/browser/devtools/";
 addRDMTask(
  null,
  async function() {
-    // Disable bfcache for Fission for now.
-    // If Fission is disabled, the pref is no-op.
-    await SpecialPowers.pushPrefEnv({
-      set: [["fission.bfcacheInParent", false]],
-    });
-
    await SpecialPowers.pushPrefEnv({
      set: [["browser.navigation.requireUserInteraction", false]],
    });
--- a/devtools/client/styleeditor/test/browser_styleeditor_fetch-from-netmonitor.js
+++ b/devtools/client/styleeditor/test/browser_styleeditor_fetch-from-netmonitor.js
@ -10,12 +10,6 @@ const EMPTY_TEST_URL = TEST_BASE_HTTP + "doc_empty.html";
 const TEST_URL = TEST_BASE_HTTP + "doc_fetch_from_netmonitor.html";

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  info("Opening netmonitor");
  // Navigate first to an empty document in order to:
  // * avoid introducing a cross process navigation when calling navigateTo()
--- a/devtools/client/webconsole/test/browser/browser_jsterm_autocomplete_getters_cache.js
+++ b/devtools/client/webconsole/test/browser/browser_jsterm_autocomplete_getters_cache.js
@ -33,12 +33,6 @@ const TEST_URI = `data:text/html;charset=utf-8,
 <body>Autocomplete popup - invoke getter cache test</body>`;

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const hud = await openNewTabAndConsole(TEST_URI);
  const { jsterm } = hud;
  const { autocompletePopup } = jsterm;
--- a/devtools/client/webconsole/test/browser/browser_webconsole_sandbox_update_after_navigation.js
+++ b/devtools/client/webconsole/test/browser/browser_webconsole_sandbox_update_after_navigation.js
@ -13,12 +13,6 @@ const TEST_URI1 = "http://example.com/" + BASE_URI;
 const TEST_URI2 = "http://example.org/" + BASE_URI;

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  await pushPref("devtools.webconsole.persistlog", false);

  const hud = await openNewTabAndConsole(TEST_URI1);
--- a/devtools/client/webconsole/test/browser/browser_webconsole_scroll.js
+++ b/devtools/client/webconsole/test/browser/browser_webconsole_scroll.js
@ -18,12 +18,6 @@ const TEST_URI = `data:text/html;charset=utf-8,<p>Web Console test for  scroll.<
  </script>
 `;
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const hud = await openNewTabAndConsole(TEST_URI);
  const { ui } = hud;
  const outputContainer = ui.outputNode.querySelector(".webconsole-output");
--- a/devtools/client/webconsole/test/browser/browser_webconsole_sourcemap_css.js
+++ b/devtools/client/webconsole/test/browser/browser_webconsole_sourcemap_css.js
@ -26,12 +26,6 @@ const PAGE_URL = `data:text/html,
 </html>`;

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  await pushPref("devtools.source-map.client-service.enabled", true);
  await pushPref("devtools.webconsole.filter.css", true);

--- a/devtools/client/webconsole/test/browser/browser_webconsole_stubs_css_message.js
+++ b/devtools/client/webconsole/test/browser/browser_webconsole_stubs_css_message.js
@ -17,12 +17,6 @@ const TEST_URI =
 const STUB_FILE = "cssMessage.js";

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const isStubsUpdate = env.get(STUBS_UPDATE_ENV) == "true";
  info(`${isStubsUpdate ? "Update" : "Check"} ${STUB_FILE}`);

--- a/devtools/shared/commands/resource/tests/browser_resources_root_node.js
+++ b/devtools/shared/commands/resource/tests/browser_resources_root_node.js
@ -14,12 +14,6 @@
 * an empty page.
 */
 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  // Open a test tab
  const tab = await addTab("data:text/html,Root Node tests");

@ -75,12 +69,6 @@ add_task(async function() {
 * Test that the watchRootNode API provides the expected node fronts.
 */
 add_task(async function testRootNodeFrontIsCorrect() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  const tab = await addTab("data:text/html,<div id=div1>");

  const { client, resourceCommand, targetCommand } = await initResourceCommand(
--- a/devtools/shared/commands/target-configuration/tests/browser_target_configuration_command_color_scheme.js
+++ b/devtools/shared/commands/target-configuration/tests/browser_target_configuration_command_color_scheme.js
@ -8,12 +8,6 @@ const TEST_DOCUMENT = "target_configuration_test_doc.sjs";
 const TEST_URI = URL_ROOT_COM_SSL + TEST_DOCUMENT;

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  info("Setup the test page with workers of all types");
  const tab = await addTab(TEST_URI);

--- a/devtools/shared/commands/target-configuration/tests/browser_target_configuration_command_touch_events.js
+++ b/devtools/shared/commands/target-configuration/tests/browser_target_configuration_command_touch_events.js
@ -8,12 +8,6 @@ const TEST_DOCUMENT = "target_configuration_test_doc.sjs";
 const TEST_URI = URL_ROOT_COM_SSL + TEST_DOCUMENT;

 add_task(async function() {
-  // Disable bfcache for Fission for now.
-  // If Fission is disabled, the pref is no-op.
-  await SpecialPowers.pushPrefEnv({
-    set: [["fission.bfcacheInParent", false]],
-  });
-
  // Disable click hold and double tap zooming as it might interfere with the test
  await pushPref("ui.click_hold_context_menus", false);
  await pushPref("apz.allow_double_tap_zooming", false);