From 9a008c400cff98fab5b2663e8913d9b1e160a6fa Mon Sep 17 00:00:00 2001 From: Yeuk Hon Wong Date: Thu, 13 Feb 2014 09:53:53 -0500 Subject: [PATCH] Bug 915824 - Move CSP tests from content/base/test into content/base/test/csp directory. r=geekboy --HG-- rename : content/base/test/file_bug548193.sjs => content/base/test/csp/file_csp_report.sjs rename : content/base/test/file_bug717511.html => content/base/test/csp/file_multi_policy_injection_bypass.html rename : content/base/test/file_bug717511.html^headers^ => content/base/test/csp/file_multi_policy_injection_bypass.html^headers^ rename : content/base/test/file_bug717511_2.html => content/base/test/csp/file_multi_policy_injection_bypass_2.html rename : content/base/test/file_bug717511_2.html^headers^ => content/base/test/csp/file_multi_policy_injection_bypass_2.html^headers^ rename : content/base/test/file_bug558431.html => content/base/test/csp/file_policyuri_async_fetch.html rename : content/base/test/file_bug558431.html^headers^ => content/base/test/csp/file_policyuri_async_fetch.html^headers^ rename : content/base/test/file_bug650386_content.sjs => content/base/test/csp/file_redirect_content.sjs rename : content/base/test/file_bug650386_report.sjs => content/base/test/csp/file_redirect_report.sjs rename : content/base/test/file_bug702439.html => content/base/test/csp/file_subframe_run_js_if_allowed.html rename : content/base/test/test_bug702439.html^headers^ => content/base/test/csp/file_subframe_run_js_if_allowed.html^headers^ rename : content/base/test/test_bug650386_redirect_301.html => content/base/test/csp/test_301_redirect.html rename : content/base/test/test_bug650386_redirect_302.html => content/base/test/csp/test_302_redirect.html rename : content/base/test/test_bug650386_redirect_303.html => content/base/test/csp/test_303_redirect.html rename : content/base/test/test_bug650386_redirect_307.html => content/base/test/csp/test_307_redirect.html rename : content/base/test/test_bug548193.html => content/base/test/csp/test_csp_report.html rename : content/base/test/test_bug717511.html => content/base/test/csp/test_multi_policy_injection_bypass.html rename : content/base/test/test_bug558431.html => content/base/test/csp/test_policyuri_async_fetch.html rename : content/base/test/test_bug702439.html => content/base/test/csp/test_subframe_run_js_if_allowed.html --- .../file_csp_report.sjs} | 1 + .../file_multi_policy_injection_bypass.html} | 3 +++ ...lti_policy_injection_bypass.html^headers^} | 0 ...file_multi_policy_injection_bypass_2.html} | 3 +++ ...i_policy_injection_bypass_2.html^headers^} | 0 .../file_policyuri_async_fetch.html} | 3 +++ .../file_policyuri_async_fetch.html^headers^} | 0 .../file_redirect_content.sjs} | 7 ++++--- .../file_redirect_report.sjs} | 3 ++- .../csp/file_subframe_run_js_if_allowed.html | 13 +++++++++++++ ..._subframe_run_js_if_allowed.html^headers^} | 0 content/base/test/csp/mochitest.ini | 19 +++++++++++++++++++ .../test_301_redirect.html} | 2 +- .../test_302_redirect.html} | 2 +- .../test_303_redirect.html} | 2 +- .../test_307_redirect.html} | 2 +- .../test_csp_report.html} | 9 ++++++--- .../test_multi_policy_injection_bypass.html} | 4 ++-- .../test_policyuri_async_fetch.html} | 9 ++++++--- .../test_subframe_run_js_if_allowed.html} | 4 ++-- content/base/test/file_bug702439.html | 11 ----------- content/base/test/mochitest.ini | 19 ------------------- 22 files changed, 68 insertions(+), 48 deletions(-) rename content/base/test/{file_bug548193.sjs => csp/file_csp_report.sjs} (93%) rename content/base/test/{file_bug717511.html => csp/file_multi_policy_injection_bypass.html} (89%) rename content/base/test/{file_bug717511.html^headers^ => csp/file_multi_policy_injection_bypass.html^headers^} (100%) rename content/base/test/{file_bug717511_2.html => csp/file_multi_policy_injection_bypass_2.html} (89%) rename content/base/test/{file_bug717511_2.html^headers^ => csp/file_multi_policy_injection_bypass_2.html^headers^} (100%) rename content/base/test/{file_bug558431.html => csp/file_policyuri_async_fetch.html} (69%) rename content/base/test/{file_bug558431.html^headers^ => csp/file_policyuri_async_fetch.html^headers^} (100%) rename content/base/test/{file_bug650386_content.sjs => csp/file_redirect_content.sjs} (86%) rename content/base/test/{file_bug650386_report.sjs => csp/file_redirect_report.sjs} (83%) create mode 100644 content/base/test/csp/file_subframe_run_js_if_allowed.html rename content/base/test/{test_bug702439.html^headers^ => csp/file_subframe_run_js_if_allowed.html^headers^} (100%) rename content/base/test/{test_bug650386_redirect_301.html => csp/test_301_redirect.html} (96%) rename content/base/test/{test_bug650386_redirect_302.html => csp/test_302_redirect.html} (96%) rename content/base/test/{test_bug650386_redirect_303.html => csp/test_303_redirect.html} (96%) rename content/base/test/{test_bug650386_redirect_307.html => csp/test_307_redirect.html} (96%) rename content/base/test/{test_bug548193.html => csp/test_csp_report.html} (94%) rename content/base/test/{test_bug717511.html => csp/test_multi_policy_injection_bypass.html} (95%) rename content/base/test/{test_bug558431.html => csp/test_policyuri_async_fetch.html} (86%) rename content/base/test/{test_bug702439.html => csp/test_subframe_run_js_if_allowed.html} (87%) delete mode 100644 content/base/test/file_bug702439.html diff --git a/content/base/test/file_bug548193.sjs b/content/base/test/csp/file_csp_report.sjs similarity index 93% rename from content/base/test/file_bug548193.sjs rename to content/base/test/csp/file_csp_report.sjs index a04392e8ad41..baf88b02d4a6 100644 --- a/content/base/test/file_bug548193.sjs +++ b/content/base/test/csp/file_csp_report.sjs @@ -1,4 +1,5 @@ // SJS file for CSP violation report test +// https://bugzilla.mozilla.org/show_bug.cgi?id=548193 function handleRequest(request, response) { var query = {}; diff --git a/content/base/test/file_bug717511.html b/content/base/test/csp/file_multi_policy_injection_bypass.html similarity index 89% rename from content/base/test/file_bug717511.html rename to content/base/test/csp/file_multi_policy_injection_bypass.html index 739b356bfefd..82eed84570a3 100644 --- a/content/base/test/file_bug717511.html +++ b/content/base/test/csp/file_multi_policy_injection_bypass.html @@ -1,4 +1,7 @@ + diff --git a/content/base/test/file_bug717511.html^headers^ b/content/base/test/csp/file_multi_policy_injection_bypass.html^headers^ similarity index 100% rename from content/base/test/file_bug717511.html^headers^ rename to content/base/test/csp/file_multi_policy_injection_bypass.html^headers^ diff --git a/content/base/test/file_bug717511_2.html b/content/base/test/csp/file_multi_policy_injection_bypass_2.html similarity index 89% rename from content/base/test/file_bug717511_2.html rename to content/base/test/csp/file_multi_policy_injection_bypass_2.html index 03a4a063e469..6f27a043fb87 100644 --- a/content/base/test/file_bug717511_2.html +++ b/content/base/test/csp/file_multi_policy_injection_bypass_2.html @@ -1,4 +1,7 @@ + diff --git a/content/base/test/file_bug717511_2.html^headers^ b/content/base/test/csp/file_multi_policy_injection_bypass_2.html^headers^ similarity index 100% rename from content/base/test/file_bug717511_2.html^headers^ rename to content/base/test/csp/file_multi_policy_injection_bypass_2.html^headers^ diff --git a/content/base/test/file_bug558431.html b/content/base/test/csp/file_policyuri_async_fetch.html similarity index 69% rename from content/base/test/file_bug558431.html rename to content/base/test/csp/file_policyuri_async_fetch.html index 915a7e3fd6a4..cf941d15475c 100644 --- a/content/base/test/file_bug558431.html +++ b/content/base/test/csp/file_policyuri_async_fetch.html @@ -1,2 +1,5 @@ + diff --git a/content/base/test/file_bug558431.html^headers^ b/content/base/test/csp/file_policyuri_async_fetch.html^headers^ similarity index 100% rename from content/base/test/file_bug558431.html^headers^ rename to content/base/test/csp/file_policyuri_async_fetch.html^headers^ diff --git a/content/base/test/file_bug650386_content.sjs b/content/base/test/csp/file_redirect_content.sjs similarity index 86% rename from content/base/test/file_bug650386_content.sjs rename to content/base/test/csp/file_redirect_content.sjs index c4f20dc4ffc2..9a6461d69bbf 100644 --- a/content/base/test/file_bug650386_content.sjs +++ b/content/base/test/csp/file_redirect_content.sjs @@ -1,6 +1,7 @@ -// SJS file for tests for bug650386, serves file_bug650386_content.html +// https://bugzilla.mozilla.org/show_bug.cgi?id=650386 +// This SJS file serves file_redirect_content.html // with a CSP that will trigger a violation and that will report it -// to file_bug650386_report.sjs +// to file_redirect_report.sjs // // This handles 301, 302, 303 and 307 redirects. The HTTP status code // returned/type of redirect to do comes from the query string @@ -22,7 +23,7 @@ function handleRequest(request, response) { return; } - var csp = "default-src \'self\';report-uri http://mochi.test:8888/tests/content/base/test/file_bug650386_report.sjs?" + redirect; + var csp = "default-src \'self\';report-uri http://mochi.test:8888/tests/content/base/test/csp/file_redirect_report.sjs?" + redirect; response.setHeader("X-Content-Security-Policy", csp, false); diff --git a/content/base/test/file_bug650386_report.sjs b/content/base/test/csp/file_redirect_report.sjs similarity index 83% rename from content/base/test/file_bug650386_report.sjs rename to content/base/test/csp/file_redirect_report.sjs index af777a9ef8b0..9cc7e6548638 100644 --- a/content/base/test/file_bug650386_report.sjs +++ b/content/base/test/csp/file_redirect_report.sjs @@ -1,4 +1,5 @@ -// SJS file for tests for bug650386, this serves as CSP violation report target +// https://bugzilla.mozilla.org/show_bug.cgi?id=650386 +// This SJS file serves as CSP violation report target // and issues a redirect, to make sure the browser does not post to the target // of the redirect, per CSP spec. // This handles 301, 302, 303 and 307 redirects. The HTTP status code diff --git a/content/base/test/csp/file_subframe_run_js_if_allowed.html b/content/base/test/csp/file_subframe_run_js_if_allowed.html new file mode 100644 index 000000000000..3ba970ce8469 --- /dev/null +++ b/content/base/test/csp/file_subframe_run_js_if_allowed.html @@ -0,0 +1,13 @@ + + + +click + + diff --git a/content/base/test/test_bug702439.html^headers^ b/content/base/test/csp/file_subframe_run_js_if_allowed.html^headers^ similarity index 100% rename from content/base/test/test_bug702439.html^headers^ rename to content/base/test/csp/file_subframe_run_js_if_allowed.html^headers^ diff --git a/content/base/test/csp/mochitest.ini b/content/base/test/csp/mochitest.ini index b80539374142..050178dd455f 100644 --- a/content/base/test/csp/mochitest.ini +++ b/content/base/test/csp/mochitest.ini @@ -109,6 +109,17 @@ support-files = file_self_none_as_hostname_confusion.html file_self_none_as_hostname_confusion.html^headers^ file_csp_testserver.sjs + file_csp_report.sjs + file_policyuri_async_fetch.html + file_policyuri_async_fetch.html^headers^ + file_redirect_content.sjs + file_redirect_report.sjs + file_subframe_run_js_if_allowed.html + file_subframe_run_js_if_allowed.html^headers^ + file_multi_policy_injection_bypass.html + file_multi_policy_injection_bypass.html^headers^ + file_multi_policy_injection_bypass_2.html + file_multi_policy_injection_bypass_2.html^headers^ file_csp_regexp_parsing.html file_csp_regexp_parsing.js @@ -136,4 +147,12 @@ support-files = [test_dual_headers_warning.html] [test_self_none_as_hostname_confusion.html] [test_bug949549.html] +[test_csp_report.html] +[test_policyuri_async_fetch.html] +[test_301_redirect.html] +[test_302_redirect.html] +[test_303_redirect.html] +[test_307_redirect.html] +[test_subframe_run_js_if_allowed.html] +[test_multi_policy_injection_bypass.html] [test_csp_regexp_parsing.html] diff --git a/content/base/test/test_bug650386_redirect_301.html b/content/base/test/csp/test_301_redirect.html similarity index 96% rename from content/base/test/test_bug650386_redirect_301.html rename to content/base/test/csp/test_301_redirect.html index d90c2c4c4b92..16fe91750999 100644 --- a/content/base/test/test_bug650386_redirect_301.html +++ b/content/base/test/csp/test_301_redirect.html @@ -71,7 +71,7 @@ window.done = function(result) { SimpleTest.waitForExplicitFinish(); // save this for last so that our listeners are registered. -document.getElementById('content_iframe').src = 'file_bug650386_content.sjs?301'; +document.getElementById('content_iframe').src = 'file_redirect_content.sjs?301'; diff --git a/content/base/test/test_bug650386_redirect_302.html b/content/base/test/csp/test_302_redirect.html similarity index 96% rename from content/base/test/test_bug650386_redirect_302.html rename to content/base/test/csp/test_302_redirect.html index b45fddb9b5c6..bbd618e34f37 100644 --- a/content/base/test/test_bug650386_redirect_302.html +++ b/content/base/test/csp/test_302_redirect.html @@ -71,7 +71,7 @@ window.done = function(result) { SimpleTest.waitForExplicitFinish(); // save this for last so that our listeners are registered. -document.getElementById('content_iframe').src = 'file_bug650386_content.sjs?302'; +document.getElementById('content_iframe').src = 'file_redirect_content.sjs?302'; diff --git a/content/base/test/test_bug650386_redirect_303.html b/content/base/test/csp/test_303_redirect.html similarity index 96% rename from content/base/test/test_bug650386_redirect_303.html rename to content/base/test/csp/test_303_redirect.html index ea0075f10c0c..40052784c512 100644 --- a/content/base/test/test_bug650386_redirect_303.html +++ b/content/base/test/csp/test_303_redirect.html @@ -71,7 +71,7 @@ window.done = function(result) { SimpleTest.waitForExplicitFinish(); // save this for last so that our listeners are registered. -document.getElementById('content_iframe').src = 'file_bug650386_content.sjs?303'; +document.getElementById('content_iframe').src = 'file_redirect_content.sjs?303'; diff --git a/content/base/test/test_bug650386_redirect_307.html b/content/base/test/csp/test_307_redirect.html similarity index 96% rename from content/base/test/test_bug650386_redirect_307.html rename to content/base/test/csp/test_307_redirect.html index d16670602a8f..a35b89e7a6b3 100644 --- a/content/base/test/test_bug650386_redirect_307.html +++ b/content/base/test/csp/test_307_redirect.html @@ -71,7 +71,7 @@ window.done = function(result) { SimpleTest.waitForExplicitFinish(); // save this for last so that our listeners are registered. -document.getElementById('content_iframe').src = 'file_bug650386_content.sjs?307'; +document.getElementById('content_iframe').src = 'file_redirect_content.sjs?307'; diff --git a/content/base/test/test_bug548193.html b/content/base/test/csp/test_csp_report.html similarity index 94% rename from content/base/test/test_bug548193.html rename to content/base/test/csp/test_csp_report.html index 94d621d569fd..1462bd85e466 100644 --- a/content/base/test/test_bug548193.html +++ b/content/base/test/csp/test_csp_report.html @@ -1,7 +1,10 @@ + - Test for CSP JSON violation report + Test for Bug 548193 @@ -73,13 +76,13 @@ examiner.prototype = { } // content file that triggers a violation report -var testFile = "file_bug548193.sjs"; +var testFile = "file_csp_report.sjs"; window.checkResults = function(reportObj) { var cspReport = reportObj["csp-report"]; // correct violating request is(cspReport["document-uri"], - "http://mochi.test:8888/tests/content/base/test/" + testFile, + "http://mochi.test:8888/tests/content/base/test/csp/" + testFile, "Incorrect violating request"); // correct blocked-uri is(cspReport["blocked-uri"], diff --git a/content/base/test/test_bug717511.html b/content/base/test/csp/test_multi_policy_injection_bypass.html similarity index 95% rename from content/base/test/test_bug717511.html rename to content/base/test/csp/test_multi_policy_injection_bypass.html index 364394f02952..d97de436590b 100644 --- a/content/base/test/test_bug717511.html +++ b/content/base/test/csp/test_multi_policy_injection_bypass.html @@ -115,8 +115,8 @@ SimpleTest.waitForExplicitFinish(); // save this for last so that our listeners are registered. // ... this loads the testbed of good and bad requests. -document.getElementById('cspframe').src = 'file_bug717511.html'; -document.getElementById('cspframe2').src = 'file_bug717511_2.html'; +document.getElementById('cspframe').src = 'file_multi_policy_injection_bypass.html'; +document.getElementById('cspframe2').src = 'file_multi_policy_injection_bypass_2.html'; diff --git a/content/base/test/test_bug558431.html b/content/base/test/csp/test_policyuri_async_fetch.html similarity index 86% rename from content/base/test/test_bug558431.html rename to content/base/test/csp/test_policyuri_async_fetch.html index cb681c877f17..c782d1a4997a 100644 --- a/content/base/test/test_bug558431.html +++ b/content/base/test/csp/test_policyuri_async_fetch.html @@ -1,8 +1,11 @@ + - Test for CSP async policy-uri - + Test for Bug 558431 + @@ -24,7 +27,7 @@ f.addEventListener('load', function() { SimpleTest.finish(); }, false); // load the test frame -f.src = "file_bug558431.html"; +f.src = "file_policyuri_async_fetch.html"; SimpleTest.waitForExplicitFinish(); diff --git a/content/base/test/test_bug702439.html b/content/base/test/csp/test_subframe_run_js_if_allowed.html similarity index 87% rename from content/base/test/test_bug702439.html rename to content/base/test/csp/test_subframe_run_js_if_allowed.html index e1fdecbf8c2d..96a70b366a94 100644 --- a/content/base/test/test_bug702439.html +++ b/content/base/test/csp/test_subframe_run_js_if_allowed.html @@ -8,7 +8,7 @@ permitted to execute javascript: URLs assuming the policy allows this. --> - Test for Bug 428847 + Test for Bug 702439 @@ -27,6 +27,6 @@ function checkResult() SimpleTest.waitForExplicitFinish(); - + diff --git a/content/base/test/file_bug702439.html b/content/base/test/file_bug702439.html deleted file mode 100644 index b679330fbac6..000000000000 --- a/content/base/test/file_bug702439.html +++ /dev/null @@ -1,11 +0,0 @@ - - - -click - - diff --git a/content/base/test/mochitest.ini b/content/base/test/mochitest.ini index 25ffa92d4325..27d17f01e135 100644 --- a/content/base/test/mochitest.ini +++ b/content/base/test/mochitest.ini @@ -94,10 +94,7 @@ support-files = file_bug503481b_inner.html file_bug541937.html file_bug541937.xhtml - file_bug548193.sjs file_bug557892.html - file_bug558431.html - file_bug558431.html^headers^ file_bug562137.txt file_bug590812-ref.xhtml file_bug590812.xml @@ -112,8 +109,6 @@ support-files = file_bug604660-6.xsl file_bug622088.sjs file_bug622088_inner.html - file_bug650386_content.sjs - file_bug650386_report.sjs file_bug675121.sjs file_bug687859-16.js file_bug687859-16.js^headers^ @@ -124,16 +119,11 @@ support-files = file_bug687859-http.js^headers^ file_bug687859-inherit.js file_bug692434.xml - file_bug702439.html file_bug707142_baseline.json file_bug707142_bom.json file_bug707142_utf-16.json file_bug708620-2.html file_bug708620.html - file_bug717511.html - file_bug717511.html^headers^ - file_bug717511_2.html - file_bug717511_2.html^headers^ file_bug782342.txt file_bug787778.sjs file_bug804395.jar @@ -411,11 +401,9 @@ support-files = [test_bug544642.html] [test_bug545644.html] [test_bug545644.xhtml] -[test_bug548193.html] [test_bug548463.html] [test_bug553896.xhtml] [test_bug557892.html] -[test_bug558431.html] [test_bug558726.html] [test_bug559526.html] [test_bug560780.html] @@ -453,10 +441,6 @@ support-files = [test_bug631615.html] [test_bug638112.html] [test_bug647518.html] -[test_bug650386_redirect_301.html] -[test_bug650386_redirect_302.html] -[test_bug650386_redirect_303.html] -[test_bug650386_redirect_307.html] [test_bug656283.html] [test_bug664916.html] [test_bug666604.html] @@ -478,14 +462,11 @@ support-files = [test_bug696301-2.html] [test_bug698381.html] [test_bug698384.html] -[test_bug702439.html] -[test_bug702439.html^headers^] [test_bug704063.html] [test_bug707142.html] [test_bug708620.html] [test_bug711047.html] [test_bug711180.html] -[test_bug717511.html] [test_bug719533.html] [test_bug726364.html] [test_bug737087.html]