Backed out changeset bc0b215802bb (bug 1785123) for causing mochitest failures on test_privateWin.html. CLOSED TREE

dom/fs/test/gtest/api/TestFileSystemDirectoryHandle.cpp

@@ -22,11 +22,6 @@ namespace mozilla::dom::fs::test {
 class TestFileSystemDirectoryHandle : public ::testing::Test {
  protected:
   void SetUp() override {
-    // TODO: Fix the test to not depend on CreateFileSystemManagerParent
-    // failure because of the pref set to false.
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", false);
-
     mRequestHandler = MakeUnique<MockFileSystemRequestHandler>();
     mMetadata = FileSystemEntryMetadata("dir"_ns, u"Directory"_ns,
                                         /* directory */ true);
@@ -35,9 +30,6 @@ class TestFileSystemDirectoryHandle : public ::testing::Test {
   }
 
   void TearDown() override {
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", true);
-
     if (!mManager->IsShutdown()) {
       mManager->Shutdown();
     }

dom/fs/test/gtest/api/TestFileSystemFileHandle.cpp

@@ -22,11 +22,6 @@ namespace mozilla::dom::fs::test {
 class TestFileSystemFileHandle : public ::testing::Test {
  protected:
   void SetUp() override {
-    // TODO: Fix the test to not depend on CreateFileSystemManagerParent
-    // failure because of the pref set to false.
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", false);
-
     mRequestHandler = MakeUnique<MockFileSystemRequestHandler>();
     mMetadata =
         FileSystemEntryMetadata("file"_ns, u"File"_ns, /* directory */ false);
@@ -34,9 +29,6 @@ class TestFileSystemFileHandle : public ::testing::Test {
   }
 
   void TearDown() override {
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", true);
-
     if (!mManager->IsShutdown()) {
       mManager->Shutdown();
     }

dom/fs/test/gtest/api/TestFileSystemHandle.cpp

@@ -20,11 +20,6 @@ namespace mozilla::dom::fs::test {
 class TestFileSystemHandle : public ::testing::Test {
  protected:
   void SetUp() override {
-    // TODO: Fix the test to not depend on CreateFileSystemManagerParent
-    // failure because of the pref set to false.
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", false);
-
     mDirMetadata = FileSystemEntryMetadata("dir"_ns, u"Directory"_ns,
                                            /* directory */ true);
     mFileMetadata =
@@ -33,9 +28,6 @@ class TestFileSystemHandle : public ::testing::Test {
   }
 
   void TearDown() override {
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", true);
-
     if (!mManager->IsShutdown()) {
       mManager->Shutdown();
     }

dom/fs/test/gtest/child/TestFileSystemBackgroundRequestHandler.cpp

@@ -18,19 +18,9 @@ namespace mozilla::dom::fs::test {
 class TestFileSystemBackgroundRequestHandler : public ::testing::Test {
  protected:
   void SetUp() override {
-    // TODO: Fix the test to not depend on CreateFileSystemManagerParent
-    // failure because of the pref set to false.
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", false);
-
     mFileSystemManagerChild = MakeAndAddRef<TestFileSystemManagerChild>();
   }
 
-  void TearDown() override {
-    nsCOMPtr<nsIPrefBranch> prefs = do_GetService(NS_PREFSERVICE_CONTRACTID);
-    prefs->SetBoolPref("dom.fs.enabled", true);
-  }
-
   RefPtr<FileSystemBackgroundRequestHandler>
   GetFileSystemBackgroundRequestHandler() {
     return MakeRefPtr<FileSystemBackgroundRequestHandler>(

dom/serviceworkers/test/test_serviceworker_interfaces.js

@@ -189,14 +189,6 @@ let interfaceNamesInGlobalScope = [
   // IMPORTANT: Do not change this list without review from a DOM peer!
   "FileReader",
   // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemDirectoryHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemFileHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemWritableFileStream", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
   "FontFace",
   // IMPORTANT: Do not change this list without review from a DOM peer!
   "FontFaceSet",

dom/tests/mochitest/general/test_interfaces.js

@@ -417,20 +417,12 @@ let interfaceNamesInGlobalScope = [
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FileSystemDirectoryEntry", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemDirectoryHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FileSystemDirectoryReader", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FileSystemEntry", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FileSystemFileEntry", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemFileHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemWritableFileStream", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FocusEvent", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FormData", insecureContext: true },

dom/workers/test/test_worker_interfaces.js

@@ -194,16 +194,6 @@ let interfaceNamesInGlobalScope = [
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FileReaderSync", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemDirectoryHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemFileHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemSyncAccessHandle", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "FileSystemWritableFileStream", nightly: true },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FontFace", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "FontFaceSet", insecureContext: true },

modules/libpref/init/StaticPrefList.yaml

@@ -3888,7 +3888,7 @@
 # Whether the File System API is enabled
 - name: dom.fs.enabled
   type: RelaxedAtomicBool
-  value: @IS_NIGHTLY_BUILD@
+  value: false
   mirror: always
 
 # Whether the WritableFileStream is enabled or disabled.