Issue 3459 - Make content policy test for fetch() explicitly dispatch abp:frameready

chrome/content/tests/policy.js

Index: chrome/content/tests/policy.js
===================================================================
--- a/chrome/content/tests/policy.js
+++ b/chrome/content/tests/policy.js
@@ -294,8 +294,19 @@
   {
     tests.push([
       "fetch() API",
-      '<script>fetch("test.xml").catch(function() {});</script>',
-      "http://127.0.0.1:1234/test.xml", "xmlhttprequest", false, false
+      '<script>' +
+        'fetch("test.xml").then(' +
+          'function()' +
+          '{' +
+            dispatchReadyEvent +
+          '},' +
+          'function()' +
+          '{' +
+            dispatchReadyEvent +
+          '}' +
+        ');' +
+      '</script>',
+      "http://127.0.0.1:1234/test.xml", "xmlhttprequest", false, true
     ]);
   }
 
@@ -454,7 +465,7 @@
 
   for (let test = 0; test < tests.length; test++)
   {
-    let [name, body, expectedURL, expectedType, expectedDomain, expectedThirdParty] = tests[test];
+    let [name, , expectedURL] = tests[test];

[Sebastian Noack, 2015/12/31 14:36:39]

I were already wondering whether the last parameter indicates async tests.
However, this line, wrongly unpacking the array, tricked me.

     for (let stage = 1; stage in stageDescriptions; stage++)
     {
       let stageDescription = stageDescriptions[stage];