Bug 1387811 - Follow up for Test within Bug 1381761: CSP JSON is neve…

…r null, hence it's better to check actual contents of JSON for testing. r=dveditz
dothq · Aug 6, 2017 · a1341cc · a1341cc
1 parent a5f2311
commit a1341cc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/dom/security/test/csp/file_data_csp_inheritance.html b/dom/security/test/csp/file_data_csp_inheritance.html
@@ -13,8 +13,10 @@
   var frame = document.getElementById("dataFrame");
   var principal = SpecialPowers.wrap(frame.contentDocument).nodePrincipal;
   var cspJSON = principal.cspJSON;
-  var result = principal.cspJSON ? "dataInheritsCSP" : "dataDoesNotInheritCSP";
-  window.parent.postMessage({result}, "*");
+  var cspOBJ = JSON.parse(principal.cspJSON);
+  // make sure we got >>one<< policy
+  var policies = cspOBJ["csp-policies"];
+  window.parent.postMessage({result: policies.length}, "*");
 </script>
 
 </body>

diff --git a/dom/security/test/csp/test_data_csp_inheritance.html b/dom/security/test/csp/test_data_csp_inheritance.html
@@ -22,7 +22,9 @@
 window.addEventListener("message", receiveMessage);
 function receiveMessage(event) {
   window.removeEventListener("message", receiveMessage);
-  is(event.data.result, "dataInheritsCSP",
+  // toplevel CSP should apply to data: URI iframe hence resulting
+  // in 1 applied policy.
+  is(event.data.result, 1,
      "data: URI iframe inherits CSP from including context");
   SimpleTest.finish();
 }