Bug 1594004 - Enable CacheSplit on default r=ckerschb,annevk

***
Fix Cache-Split related Tests

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

--HG--
extra : moz-landing-system : lando

browser/components/originattributes/test/browser/browser_cache.js

@@ -151,6 +151,7 @@ async function doInit(aMode) {
     set: [
       ["network.predictor.enabled", false],
       ["network.predictor.enable-prefetch", false],
+      ["browser.cache.cache_isolation", false],
     ],
   });
   clearAllImageCaches();

browser/components/originattributes/test/browser/browser_imageCacheIsolation.js

@@ -19,6 +19,8 @@ server.start(-1);
 // Disable rcwn to make cache behavior deterministic.
 let rcwnEnabled = Services.prefs.getBoolPref("network.http.rcwn.enabled");
 Services.prefs.setBoolPref("network.http.rcwn.enabled", false);
+// Disable additional Http cache isolation.
+Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
 
 registerCleanupFunction(() => {
   Services.prefs.setBoolPref("network.http.rcwn.enabled", rcwnEnabled);

modules/libpref/init/StaticPrefList.yaml

@@ -687,7 +687,12 @@
 # Enable/Disable Origin based cache isolation
 - name: browser.cache.cache_isolation
   type: RelaxedAtomicBool
+
+#if defined(NIGHTLY_BUILD)
+  value: true
+#else
   value: false
+#endif
   mirror: always
 
 # -1 = determine dynamically, 0 = none, n = memory capacity in kilobytes.

netwerk/test/browser/browser_about_cache.js

@@ -4,6 +4,9 @@
  * Open a dummy page, then open about:cache and verify the opened page shows up in the cache.
  */
 add_task(async function() {
+  await SpecialPowers.pushPrefEnv({
+    set: [["browser.cache.cache_isolation", false]],
+  });
   const kRoot = getRootDirectory(gTestPath).replace(
     "chrome://mochitests/content/",
     "https://example.com/"

netwerk/test/unit/test_bug1064258.js

@@ -53,6 +53,8 @@ function contentHandler2(metadata, response) {
 }
 
 function run_test() {
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
+
   httpServer = new HttpServer();
   httpServer.registerPathHandler("/content1", contentHandler1);
   httpServer.registerPathHandler("/content2", contentHandler2);

netwerk/test/unit/test_bug482601.js

@@ -37,6 +37,8 @@ var listener = {
 };
 
 function run_test() {
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
+
   httpserv = new HttpServer();
   httpserv.registerPathHandler("/bug482601/nocache", bug482601_nocache);
   httpserv.registerPathHandler("/bug482601/partial", bug482601_partial);

netwerk/test/unit/test_bug482934.js

@@ -9,7 +9,6 @@ var request_time;
 var response_time;
 
 var cache_storage;
-
 var httpserver = new HttpServer();
 httpserver.start(-1);
 
@@ -69,6 +68,8 @@ function check_cached_data(aCachedData, aCallback) {
 }
 
 function run_test() {
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
+
   do_get_profile();
   evict_cache_entries();
 

netwerk/test/unit/test_bug618835.js

@@ -80,6 +80,8 @@ FinalListener.prototype = {
 };
 
 function run_test() {
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
+
   httpserv = new HttpServer();
   httpserv.registerPathHandler("/cl", content_location);
   httpserv.registerPathHandler("/post", post_target);

netwerk/test/unit/test_bug812167.js

@@ -127,6 +127,7 @@ function finish_test() {
 
 function run_test() {
   do_get_profile();
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
 
   httpserver = new HttpServer();
   httpserver.registerPathHandler(randomPath1, redirectHandler_NoStore);

netwerk/test/unit/test_cache-control_request.js

@@ -79,6 +79,7 @@ function run_test() {
   do_test_pending();
 
   Services.prefs.setBoolPref("network.http.rcwn.enabled", false);
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
 
   httpserver.registerPathHandler(resource_age_100, resource_age_100_handler);
   httpserver.registerPathHandler(

netwerk/test/unit/test_inhibit_caching.js

@@ -19,6 +19,7 @@ var httpserver = null;
 
 function run_test() {
   // setup test
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
   httpserver = new HttpServer();
   httpserver.registerPathHandler("/test", contentHandler);
   httpserver.start(-1);

netwerk/test/unit/test_predictor.js

@@ -739,6 +739,7 @@ function run_test_real() {
   tests.forEach(f => add_test(f));
   do_get_profile();
 
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
   Services.prefs.setBoolPref("network.predictor.enabled", true);
   Services.prefs.setBoolPref("network.predictor.cleaned-up", true);
   Services.prefs.setBoolPref("network.predictor.doing-tests", true);

netwerk/test/unit/test_suspend_channel_on_examine_merged_response.js

@@ -55,6 +55,7 @@ var listener = {
 };
 
 function run_test() {
+  Services.prefs.setBoolPref("browser.cache.cache_isolation", false);
   httpserv = new HttpServer();
   httpserv.registerPathHandler("/path/partial", path_partial);
   httpserv.registerPathHandler("/path/cached", path_cached);

testing/web-platform/meta/fetch/http-cache/invalidate.html.ini

@@ -1,4 +1,5 @@
 [invalidate.html]
+  prefs: [browser.cache.cache_isolation:false]
   [HTTP cache does not invalidate after a failed response from an unsafe request]
     expected: FAIL
 

toolkit/components/antitracking/test/browser/browser_networkIsolation.js

@@ -37,6 +37,7 @@ add_task(async function() {
     set: [
       ["browser.cache.disk.enable", false],
       ["browser.cache.memory.enable", false],
+      ["browser.cache.cache_isolation", false],
       [
         "network.cookie.cookieBehavior",
         Ci.nsICookieService.BEHAVIOR_REJECT_TRACKER,