Bug 1291024 - Avoid caching safebrowsing testcase resources. r=francois

MozReview-Commit-ID: 9SB5diTrr5F

--HG--
extra : rebase_source : 9bd5f644f46627bd00871f58111bda54732df4e4

toolkit/components/url-classifier/tests/mochitest/bad.css^headers^

@@ -0,0 +1 @@
+Cache-Control: no-store

toolkit/components/url-classifier/tests/mochitest/evil.css^headers^

@@ -0,0 +1 @@
+Cache-Control: no-store

toolkit/components/url-classifier/tests/mochitest/evil.js^headers^

@@ -1 +1,2 @@
 Access-Control-Allow-Origin: *
+Cache-Control: no-store

toolkit/components/url-classifier/tests/mochitest/mochitest.ini

@@ -8,6 +8,7 @@ support-files =
   cleanWorker.js
   good.js
   evil.css
+  evil.css^headers^
   evil.js
   evil.js^headers^
   evilWorker.js
@@ -24,6 +25,7 @@ support-files =
   dnt.sjs
   update.sjs
   bad.css
+  bad.css^headers^
   gethash.sjs
   gethashFrame.html
 

toolkit/components/url-classifier/tests/mochitest/test_bug1254766.html

@@ -289,8 +289,13 @@ function runTest() {
 }
 
 SimpleTest.waitForExplicitFinish();
+
+// 'network.predictor.enabled' is disabled because if other testcase load
+// evil.js, evil.css ...etc resources, it may cause we load them from cache
+// directly and bypass classifier check
 SpecialPowers.pushPrefEnv({"set": [
-  ["browser.safebrowsing.malware.enabled", true]
+  ["browser.safebrowsing.malware.enabled", true],
+  ["network.predictor.enabled", false],
 ]}, runTest);
 
 </script>

toolkit/components/url-classifier/tests/mochitest/test_gethash.html

@@ -141,8 +141,13 @@ function runTest() {
 }
 
 SimpleTest.waitForExplicitFinish();
+
+// 'network.predictor.enabled' is disabled because if other testcase load
+// evil.js, evil.css ...etc resources, it may cause we load them from cache
+// directly and bypass classifier check
 SpecialPowers.pushPrefEnv({"set": [
-  ["browser.safebrowsing.malware.enabled", true]
+  ["browser.safebrowsing.malware.enabled", true],
+  ["network.predictor.enabled", false],
 ]}, runTest);
 
 </script>