From 3b693fede558092136ddc9270602db9721b2cbe3 Mon Sep 17 00:00:00 2001 From: Alexandre Poirot Date: Wed, 25 Aug 2021 17:16:27 +0000 Subject: [PATCH] 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 --- .../browser_application_panel_list-domain-workers.js | 6 ------ ...browser_application_panel_list-several-workers.js | 6 ------ .../test/mochitest/browser_dbg-link-reload.js | 6 ------ .../test/mochitest/browser_dbg-preview-module.js | 6 ------ .../test/mochitest/browser_dbg-watchpoints.js | 6 ------ .../test/browser_toolbox_fission_navigation.js | 6 ------ .../test/browser_grids_restored-after-reload.js | 6 ------ ...ser_grids_restored-multiple-grids-after-reload.js | 6 ------ .../test/browser_inspector_highlighter-05.js | 6 ------ .../browser_inspector_highlighter-eyedropper-xul.js | 6 ------ .../inspector/test/browser_inspector_startup.js | 6 ------ .../browser/browser_memory_fission_switch_target.js | 6 ------ .../netmonitor/test/browser_net_block-context.js | 6 ------ .../netmonitor/test/browser_net_block-draganddrop.js | 6 ------ devtools/client/netmonitor/test/browser_net_block.js | 6 ------ .../netmonitor/test/browser_net_cause_source_map.js | 6 ------ .../client/netmonitor/test/browser_net_charts-01.js | 6 ------ .../client/netmonitor/test/browser_net_charts-02.js | 6 ------ .../client/netmonitor/test/browser_net_charts-03.js | 6 ------ .../client/netmonitor/test/browser_net_charts-04.js | 6 ------ .../client/netmonitor/test/browser_net_charts-05.js | 6 ------ .../client/netmonitor/test/browser_net_charts-06.js | 6 ------ .../client/netmonitor/test/browser_net_charts-07.js | 6 ------ .../netmonitor/test/browser_net_columns_showhide.js | 6 ------ .../test/browser_net_edit_resend_cancel.js | 6 ------ .../netmonitor/test/browser_net_edit_resend_caret.js | 6 ------ .../client/netmonitor/test/browser_net_initiator.js | 6 ------ .../test/browser_net_telemetry_edit_resend.js | 6 ------ .../test/browser_net_telemetry_filters_changed.js | 6 ------ .../test/browser_net_telemetry_sidepanel_changed.js | 6 ------ .../netmonitor/test/browser_net_websocket_stacks.js | 6 ------ .../netmonitor/test/browser_net_worker_stacks.js | 6 ------ .../responsive/test/browser/browser_navigation.js | 6 ------ .../browser_styleeditor_fetch-from-netmonitor.js | 6 ------ .../browser_jsterm_autocomplete_getters_cache.js | 6 ------ ...ser_webconsole_sandbox_update_after_navigation.js | 6 ------ .../test/browser/browser_webconsole_scroll.js | 6 ------ .../test/browser/browser_webconsole_sourcemap_css.js | 6 ------ .../browser/browser_webconsole_stubs_css_message.js | 6 ------ .../resource/tests/browser_resources_root_node.js | 12 ------------ ...wser_target_configuration_command_color_scheme.js | 6 ------ ...wser_target_configuration_command_touch_events.js | 6 ------ 42 files changed, 258 deletions(-) diff --git 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 index f8c88b72a63c..28c037434520 100644 --- 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( diff --git 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 index eeff30607091..931333d392e0 100644 --- 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( diff --git a/devtools/client/debugger/test/mochitest/browser_dbg-link-reload.js b/devtools/client/debugger/test/mochitest/browser_dbg-link-reload.js index 8364046aedd7..8e99df4b3f09 100644 --- 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 }, diff --git a/devtools/client/debugger/test/mochitest/browser_dbg-preview-module.js b/devtools/client/debugger/test/mochitest/browser_dbg-preview-module.js index eaff6f7e4662..6688dbfb47e2 100644 --- 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"); diff --git a/devtools/client/debugger/test/mochitest/browser_dbg-watchpoints.js b/devtools/client/debugger/test/mochitest/browser_dbg-watchpoints.js index 76dba14daa94..8d886aebca23 100644 --- 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 diff --git a/devtools/client/framework/test/browser_toolbox_fission_navigation.js b/devtools/client/framework/test/browser_toolbox_fission_navigation.js index 53676e91bc5d..c228df216a79 100644 --- 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=
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(); diff --git a/devtools/client/inspector/grids/test/browser_grids_restored-after-reload.js b/devtools/client/inspector/grids/test/browser_grids_restored-after-reload.js index 3c350f881332..2f85785f0201 100644 --- 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; diff --git 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 index b3fe69025e0b..7cf977164ffa 100644 --- 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(); diff --git a/devtools/client/inspector/test/browser_inspector_highlighter-05.js b/devtools/client/inspector/test/browser_inspector_highlighter-05.js index 1381de42651d..c07eb87c9907 100644 --- 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"); diff --git a/devtools/client/inspector/test/browser_inspector_highlighter-eyedropper-xul.js b/devtools/client/inspector/test/browser_inspector_highlighter-eyedropper-xul.js index 2ddea6082eff..eaf2ab754a65 100644 --- 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,

HTML test 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]], - }); - const { inspector } = await openInspectorForURL(TEST_URL); info("Check the inspector toolbar"); diff --git a/devtools/client/inspector/test/browser_inspector_startup.js b/devtools/client/inspector/test/browser_inspector_startup.js index dbb8ef07db60..d8204dfc26ab 100644 --- 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 = ""; 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"); diff --git a/devtools/client/memory/test/browser/browser_memory_fission_switch_target.js b/devtools/client/memory/test/browser/browser_memory_fission_switch_target.js index 26bdfa9684c3..dc4deb7af886 100644 --- 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); diff --git a/devtools/client/netmonitor/test/browser_net_block-context.js b/devtools/client/netmonitor/test/browser_net_block-context.js index 382182a944ab..fc247c69bb64 100644 --- 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, }); diff --git a/devtools/client/netmonitor/test/browser_net_block-draganddrop.js b/devtools/client/netmonitor/test/browser_net_block-draganddrop.js index 106b7362315f..0dfe1f467b30 100644 --- 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 = diff --git a/devtools/client/netmonitor/test/browser_net_block.js b/devtools/client/netmonitor/test/browser_net_block.js index 44225e04106a..299d6fb0c304 100644 --- 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, }); diff --git a/devtools/client/netmonitor/test/browser_net_cause_source_map.js b/devtools/client/netmonitor/test/browser_net_cause_source_map.js index 2b7132fe91d4..779f354e3081 100644 --- 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 diff --git a/devtools/client/netmonitor/test/browser_net_charts-01.js b/devtools/client/netmonitor/test/browser_net_charts-01.js index e3165a9d7371..7d674493d997 100644 --- 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... "); diff --git a/devtools/client/netmonitor/test/browser_net_charts-02.js b/devtools/client/netmonitor/test/browser_net_charts-02.js index 346b98b4b275..d5c0d6fc9624 100644 --- 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 }); diff --git a/devtools/client/netmonitor/test/browser_net_charts-03.js b/devtools/client/netmonitor/test/browser_net_charts-03.js index 69589f047f55..acabcddc7dbc 100644 --- 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 }); diff --git a/devtools/client/netmonitor/test/browser_net_charts-04.js b/devtools/client/netmonitor/test/browser_net_charts-04.js index c6eaf8363441..74f6d88b0270 100644 --- 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 }); diff --git a/devtools/client/netmonitor/test/browser_net_charts-05.js b/devtools/client/netmonitor/test/browser_net_charts-05.js index 708a8f14355c..83b53e68cd1f 100644 --- 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 }); diff --git a/devtools/client/netmonitor/test/browser_net_charts-06.js b/devtools/client/netmonitor/test/browser_net_charts-06.js index 9620a1d57654..20fab19d51be 100644 --- 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 }); diff --git a/devtools/client/netmonitor/test/browser_net_charts-07.js b/devtools/client/netmonitor/test/browser_net_charts-07.js index 5020020e137c..09dd4c466354 100644 --- 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 }); diff --git a/devtools/client/netmonitor/test/browser_net_columns_showhide.js b/devtools/client/netmonitor/test/browser_net_columns_showhide.js index 0c13b7150046..9fb6167b8f2b 100644 --- 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... "); diff --git a/devtools/client/netmonitor/test/browser_net_edit_resend_cancel.js b/devtools/client/netmonitor/test/browser_net_edit_resend_cancel.js index 61625ee915c8..de7afd6e5f5d 100644 --- 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... "); diff --git a/devtools/client/netmonitor/test/browser_net_edit_resend_caret.js b/devtools/client/netmonitor/test/browser_net_edit_resend_caret.js index 9df5e8e20307..d6d46e1b6ea8 100644 --- 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... "); diff --git a/devtools/client/netmonitor/test/browser_net_initiator.js b/devtools/client/netmonitor/test/browser_net_initiator.js index c23148296a1f..4ae34d172bd4 100644 --- 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 diff --git a/devtools/client/netmonitor/test/browser_net_telemetry_edit_resend.js b/devtools/client/netmonitor/test/browser_net_telemetry_edit_resend.js index d2b4b8818165..ef3870b12f60 100644 --- 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... "); diff --git a/devtools/client/netmonitor/test/browser_net_telemetry_filters_changed.js b/devtools/client/netmonitor/test/browser_net_telemetry_filters_changed.js index 78592935e3eb..34a5dbd4b45d 100644 --- 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... "); diff --git a/devtools/client/netmonitor/test/browser_net_telemetry_sidepanel_changed.js b/devtools/client/netmonitor/test/browser_net_telemetry_sidepanel_changed.js index 8d8bfb35bfb2..e516908a2753 100644 --- 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... "); diff --git a/devtools/client/netmonitor/test/browser_net_websocket_stacks.js b/devtools/client/netmonitor/test/browser_net_websocket_stacks.js index 1bdab53d63b9..e4e4c2cf515d 100644 --- 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 }); diff --git a/devtools/client/netmonitor/test/browser_net_worker_stacks.js b/devtools/client/netmonitor/test/browser_net_worker_stacks.js index 3b16d1403cd3..de82ccc033de 100644 --- 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 }); diff --git a/devtools/client/responsive/test/browser/browser_navigation.js b/devtools/client/responsive/test/browser/browser_navigation.js index 84bd29a4d340..3c7b553e3d57 100644 --- 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]], }); diff --git a/devtools/client/styleeditor/test/browser_styleeditor_fetch-from-netmonitor.js b/devtools/client/styleeditor/test/browser_styleeditor_fetch-from-netmonitor.js index a2ac36eeccfb..cf57c47ccea2 100644 --- 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() diff --git a/devtools/client/webconsole/test/browser/browser_jsterm_autocomplete_getters_cache.js b/devtools/client/webconsole/test/browser/browser_jsterm_autocomplete_getters_cache.js index 538fce6ff6c2..d2fed8f1cb71 100644 --- 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, Autocomplete popup - invoke getter cache test`; 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; diff --git 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 index 42298f21d775..eb23f60b41e9 100644 --- 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); diff --git a/devtools/client/webconsole/test/browser/browser_webconsole_scroll.js b/devtools/client/webconsole/test/browser/browser_webconsole_scroll.js index e89572793134..e3072a2f114c 100644 --- 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,

Web Console test for scroll.< `; 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"); diff --git a/devtools/client/webconsole/test/browser/browser_webconsole_sourcemap_css.js b/devtools/client/webconsole/test/browser/browser_webconsole_sourcemap_css.js index 0fa52994f8ce..b1f219ff3209 100644 --- 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, `; 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); diff --git a/devtools/client/webconsole/test/browser/browser_webconsole_stubs_css_message.js b/devtools/client/webconsole/test/browser/browser_webconsole_stubs_css_message.js index eb5abc444cea..cfc99b31e9d6 100644 --- 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}`); diff --git a/devtools/shared/commands/resource/tests/browser_resources_root_node.js b/devtools/shared/commands/resource/tests/browser_resources_root_node.js index 8ca953b96517..8b4cdf69a9c7 100644 --- 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,

"); const { client, resourceCommand, targetCommand } = await initResourceCommand( diff --git 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 index 6c0bdb35ea6b..415c58100af1 100644 --- 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); diff --git 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 index 407e580c35f0..98d603e8b42f 100644 --- 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);