Bug 1853841 - Remove dom.storageManager.enabled pref r=webidl,smaug,dom-storage-reviewers,janv

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

dom/cache/test/mochitest/test_cache_orphaned_body.html

@@ -88,8 +88,7 @@ function gc() {
 SimpleTest.waitForExplicitFinish();
 SpecialPowers.pushPrefEnv({
   "set": [["dom.caches.testing.enabled", true],
-          ["dom.quotaManager.testing", true],
-          ["dom.storageManager.enabled", true]]
+          ["dom.quotaManager.testing", true]]
 }, function() {
   var name = "orphanedBodyOwner";
   var cache = null;

dom/indexedDB/test/unit/test_storage_manager_estimate.js

@@ -85,10 +85,5 @@ async function setup(isXOrigin) {
     );
     await SpecialPowers.wrap(document).requestStorageAccess();
   }
-  SpecialPowers.pushPrefEnv(
-    {
-      set: [["dom.storageManager.enabled", true]],
-    },
-    runTest
-  );
+  runTest();
 }

dom/quota/test/common/browser.js

@@ -23,7 +23,6 @@ function addTest(testFunction) {
 async function enableStorageTesting() {
   const prefsToSet = [
     ["dom.quotaManager.testing", true],
-    ["dom.storageManager.enabled", true],
     ["dom.simpleDB.enabled", true],
   ];
   if (Services.appinfo.OS === "WINNT") {

dom/quota/test/common/mochitest.js

@@ -8,7 +8,6 @@ loadScript("dom/quota/test/common/content.js");
 async function enableStorageTesting() {
   let prefsToSet = [
     ["dom.quotaManager.testing", true],
-    ["dom.storageManager.enabled", true],
     ["dom.simpleDB.enabled", true],
   ];
   if (SpecialPowers.Services.appinfo.OS === "WINNT") {

dom/quota/test/common/xpcshell.js

@@ -7,7 +7,6 @@ loadScript("dom/quota/test/common/system.js");
 
 function enableStorageTesting() {
   Services.prefs.setBoolPref("dom.quotaManager.testing", true);
-  Services.prefs.setBoolPref("dom.storageManager.enabled", true);
   Services.prefs.setBoolPref("dom.simpleDB.enabled", true);
   if (Services.appinfo.OS === "WINNT") {
     Services.prefs.setBoolPref("dom.quotaManager.useDOSDevicePathSyntax", true);
@@ -16,7 +15,6 @@ function enableStorageTesting() {
 
 function resetStorageTesting() {
   Services.prefs.clearUserPref("dom.quotaManager.testing");
-  Services.prefs.clearUserPref("dom.storageManager.enabled");
   Services.prefs.clearUserPref("dom.simpleDB.enabled");
   if (Services.appinfo.OS === "WINNT") {
     Services.prefs.clearUserPref("dom.quotaManager.useDOSDevicePathSyntax");

dom/quota/test/modules/system/StorageUtils.sys.mjs

@@ -29,11 +29,7 @@ export function setStoragePrefs(optionalPrefsToSet) {
 }
 
 export function clearStoragePrefs(optionalPrefsToClear) {
-  const prefsToClear = [
-    "dom.quotaManager.testing",
-    "dom.simpleDB.enabled",
-    "dom.storageManager.enabled",
-  ];
+  const prefsToClear = ["dom.quotaManager.testing", "dom.simpleDB.enabled"];
 
   if (Services.appinfo.OS === "WINNT") {
     prefsToClear.push("dom.quotaManager.useDOSDevicePathSyntax");

dom/webidl/Navigator.webidl

@@ -92,7 +92,6 @@ interface mixin NavigatorContentUtils {
 
 [SecureContext]
 interface mixin NavigatorStorage {
-  [Pref="dom.storageManager.enabled"]
   readonly attribute StorageManager storage;
 };
 

dom/webidl/StorageManager.webidl

@@ -8,9 +8,7 @@
  *
  */
 
-[SecureContext,
- Exposed=(Window,Worker),
- Pref="dom.storageManager.enabled"]
+[SecureContext, Exposed=(Window,Worker)]
 interface StorageManager {
   [NewObject]
   Promise<boolean> persisted();

modules/libpref/init/StaticPrefList.yaml

@@ -3972,12 +3972,6 @@
   value: true
   mirror: always
 
-# Whether the Storage API is enabled.
-- name: dom.storageManager.enabled
-  type: RelaxedAtomicBool
-  value: true
-  mirror: always
-
 # Whether the File System API is enabled
 - name: dom.fs.enabled
   type: RelaxedAtomicBool

testing/web-platform/meta/storage/__dir__.ini

@@ -1,2 +1 @@
-prefs: [dom.storageManager.enabled:true]
 leak-threshold: [default:51200]

testing/web-platform/mozilla/meta/dom/fs/fs-writable_unlocked_on_tab_close.https.window.js.ini

@@ -1,5 +1,3 @@
 [fs-writable_unlocked_on_tab_close.https.window.html]
   type: testharness
-  prefs:
-    [dom.storageManager.enabled:true,
-     dom.fs.enabled:true]
+  prefs: [dom.fs.enabled:true]

toolkit/components/extensions/test/mochitest/test_ext_storage_manager_capabilities.html

@@ -15,7 +15,6 @@
 add_task(async function setup() {
   await SpecialPowers.pushPrefEnv({
     "set": [
-      ["dom.storageManager.enabled", true],
       ["dom.storageManager.prompt.testing", true],
       ["dom.storageManager.prompt.testing.allow", true],
     ],

toolkit/components/extensions/test/mochitest/test_ext_unlimitedStorage.html

@@ -19,7 +19,6 @@
 async function test_background_storagePersist(EXTENSION_ID) {
   await SpecialPowers.pushPrefEnv({
     "set": [
-      ["dom.storageManager.enabled", true],
       ["dom.storageManager.prompt.testing", false],
       ["dom.storageManager.prompt.testing.allow", false],
     ],