tor-browser

commit 720721ad985538a0f5e3416cfa62b236234199b7
parent b3002681350a454c95875f97bc251c1352ddb2e2
Author: kycn <35106533+kycn@users.noreply.github.com>
Date:   Fri,  5 Dec 2025 08:27:01 +0000

Bug 2003881 - Add warning in StaticPrefList.yaml and in wpt runner about keeping ship-pref defaults in sync with the test harnesses behavior. r=ohall

Differential Revision: https://phabricator.services.mozilla.com/D274976

Diffstat:
M
modules/libpref/init/StaticPrefList.yaml
 | 
5
+++++
M
testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox_android.py
 | 
7
+++++++

2 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/modules/libpref/init/StaticPrefList.yaml b/modules/libpref/init/StaticPrefList.yaml
@@ -6142,6 +6142,11 @@
 
 # Disable storing the session history in the parent process, and accessing it
 # over IPC from the child processes.
+# WARNING: Until the test harness can understand ship-pref default values, this pref’s
+# default must be synchronized with the values set in testing/mochitest/runtests.py and in
+# testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox_android.py.
+# Changing the default here without updating the test harnesses will cause mismatches
+# in automation.
 - name: fission.disableSessionHistoryInParent
   type: bool
   value: false
diff --git a/testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox_android.py b/testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox_android.py
@@ -167,6 +167,13 @@ class ProfileCreator(FirefoxProfileCreator):
 
     @staticmethod
     def default_prefs():
+        # Until the test harness can understand default pref values,
+        # (https://bugzilla.mozilla.org/show_bug.cgi?id=1577912) this value
+        # should by synchronized with the default pref value indicated in
+        # StaticPrefList.yaml.
+        #
+        # Currently for automation, the pref defaults to false (but can be
+        # overridden with --setpref).
         return {"fission.disableSessionHistoryInParent": False}
 
     def _set_required_prefs(self, profile):