1 files changed, 188 insertions, 0 deletions
diff --git a/devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js b/devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js
new file mode 100644
index 0000000000..7350c2601c
--- /dev/null
+++ b/devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js
@@ -0,0 +1,188 @@
+/* Any copyright is dedicated to the Public Domain.
+ * http://creativecommons.org/publicdomain/zero/1.0/ */
+
+"use strict";
+
+// The test can take a while to run
+requestLongerTimeout(3);
+
+const FILENAME = "doc_backward_forward_navigation.html";
+const TEST_URI_ORG = `${URL_ROOT_ORG_SSL}${FILENAME}`;
+const TEST_URI_COM = `${URL_ROOT_COM_SSL}${FILENAME}`;
+
+Services.scriptloader.loadSubScript(
+  "chrome://mochitests/content/browser/devtools/client/debugger/test/mochitest/shared-head.js",
+  this
+);
+
+add_task(async function testMultipleNavigations() {
+  // Disable bfcache for Fission for now.
+  // If Fission is disabled, the pref is no-op.
+  await SpecialPowers.pushPrefEnv({
+    set: [["fission.bfcacheInParent", false]],
+  });
+
+  info(
+    "Test that DevTools works fine after multiple backward/forward navigations"
+  );
+  // Don't show the third panel to limit the logs and activity.
+  await pushPref("devtools.inspector.three-pane-enabled", false);
+  await pushPref("devtools.inspector.activeSidebar", "ruleview");
+  const DATA_URL = `data:text/html,<meta charset=utf8>`;
+  const tab = await addTab(DATA_URL);
+
+  // Select the debugger so there will be more activity
+  const toolbox = await openToolboxForTab(tab, "jsdebugger");
+  const inspector = await toolbox.selectTool("inspector");
+
+  info("Navigate to the ORG test page");
+  // We don't use `navigateTo` as the page is adding stylesheets and js files which might
+  // delay the load event indefinitely (and we don't need for anything to be loaded, or
+  // ready, just to register the initial navigation so we can go back and forth between urls)
+  let onLocationChange = BrowserTestUtils.waitForLocationChange(
+    gBrowser,
+    TEST_URI_ORG
+  );
+  BrowserTestUtils.startLoadingURIString(gBrowser, TEST_URI_ORG);
+  await onLocationChange;
+
+  info("And then navigate to a different origin");
+  onLocationChange = BrowserTestUtils.waitForLocationChange(
+    gBrowser,
+    TEST_URI_COM
+  );
+  BrowserTestUtils.startLoadingURIString(gBrowser, TEST_URI_COM);
+  await onLocationChange;
+
+  info(
+    "Navigate backward and forward multiple times between the two origins, with different delays"
+  );
+  await navigateBackAndForth(TEST_URI_ORG, TEST_URI_COM);
+
+  // Navigate one last time to a document with less activity so we don't have to deal
+  // with pending promises when we destroy the toolbox
+  const onInspectorReloaded = inspector.once("reloaded");
+  info("Navigate to final document");
+  await navigateTo(`${TEST_URI_ORG}?no-mutation`);
+  info("Waiting for inspector to reload…");
+  await onInspectorReloaded;
+  info("-> inspector reloaded");
+  await checkToolboxState(toolbox);
+});
+
+add_task(async function testSingleBackAndForthInstantNavigation() {
+  // Disable bfcache for Fission for now.
+  // If Fission is disabled, the pref is no-op.
+  await SpecialPowers.pushPrefEnv({
+    set: [["fission.bfcacheInParent", false]],
+  });
+
+  info(
+    "Test that DevTools works fine after navigating backward and forward right after"
+  );
+
+  // Don't show the third panel to limit the logs and activity.
+  await pushPref("devtools.inspector.three-pane-enabled", false);
+  await pushPref("devtools.inspector.activeSidebar", "ruleview");
+  const DATA_URL = `data:text/html,<meta charset=utf8>`;
+  const tab = await addTab(DATA_URL);
+
+  // Select the debugger so there will be more activity
+  const toolbox = await openToolboxForTab(tab, "jsdebugger");
+  const inspector = await toolbox.selectTool("inspector");
+
+  info("Navigate to a different origin");
+  await navigateTo(TEST_URI_COM);
+
+  info("Then navigate back, and forth immediatly");
+  // We can't call goBack and right away goForward as goForward and even the call to navigateTo
+  // a bit later might be ignored. So we wait at least for the location to change.
+  await safelyGoBack(DATA_URL);
+  await safelyGoForward(TEST_URI_COM);
+
+  // Navigate one last time to a document with less activity so we don't have to deal
+  // with pending promises when we destroy the toolbox
+  const onInspectorReloaded = inspector.once("reloaded");
+  info("Navigate to final document");
+  await navigateTo(`${TEST_URI_ORG}?no-mutation`);
+  info("Waiting for inspector to reload…");
+  await onInspectorReloaded;
+  info("-> inspector reloaded");
+  await checkToolboxState(toolbox);
+});
+
+async function checkToolboxState(toolbox) {
+  info("Check that the toolbox toolbar is still visible");
+  const toolboxTabsEl = toolbox.doc.querySelector(".toolbox-tabs");
+  ok(toolboxTabsEl, "Toolbar is still visible");
+
+  info(
+    "Check that the markup view is rendered correctly and elements can be selected"
+  );
+  const inspector = await toolbox.selectTool("inspector");
+  await waitFor(
+    () =>
+      inspector.markup &&
+      inspector.markup.win.document.body.innerText.includes(
+        `<body class="no-mutation">`
+      ),
+    `wait for <body class="no-mutation"> to be displayed in the markup view, got: ${inspector.markup?.win.document.body.innerText}`,
+    100,
+    100
+  );
+  ok(true, "the markup view is still rendered fine");
+  await selectNode("ul.logs", inspector);
+  ok(true, "Nodes can be selected");
+
+  info("Check that the debugger has some sources");
+  const dbgPanel = await toolbox.selectTool("jsdebugger");
+  const dbg = createDebuggerContext(toolbox);
+
+  info(`Wait for ${FILENAME} to be displayed in the debugger source panel`);
+  const rootNode = await waitFor(() =>
+    dbgPanel.panelWin.document.querySelector(selectors.sourceTreeRootNode)
+  );
+  await expandAllSourceNodes(dbg, rootNode);
+  const sourcesTreeScriptNode = await waitFor(() =>
+    findSourceNodeWithText(dbg, FILENAME)
+  );
+
+  ok(
+    sourcesTreeScriptNode.innerText.includes(FILENAME),
+    "The debugger has the expected source"
+  );
+}
+
+async function navigateBackAndForth(
+  expectedUrlAfterBackwardNavigation,
+  expectedUrlAfterForwardNavigation
+) {
+  const delays = [100, 0, 500];
+  for (const delay of delays) {
+    // For each delays, do 3 back/forth navigations
+    for (let i = 0; i < 3; i++) {
+      await safelyGoBack(expectedUrlAfterBackwardNavigation);
+      await wait(delay);
+      await safelyGoForward(expectedUrlAfterForwardNavigation);
+      await wait(delay);
+    }
+  }
+}
+
+async function safelyGoBack(expectedUrl) {
+  const onLocationChange = BrowserTestUtils.waitForLocationChange(
+    gBrowser,
+    expectedUrl
+  );
+  gBrowser.goBack();
+  await onLocationChange;
+}
+
+async function safelyGoForward(expectedUrl) {
+  const onLocationChange = BrowserTestUtils.waitForLocationChange(
+    gBrowser,
+    expectedUrl
+  );
+  gBrowser.goForward();
+  await onLocationChange;
+}