Bug 1011211 - Remove unneeded QI check in CSP redirect tests r=sstamm
authorGarrett Robinson <grobinson@mozilla.com>
Fri, 06 Jun 2014 11:25:54 -0700
changeset 207619 dea1f56f3feb0fa437a9449ef8907455af235056
parent 207618 77f2f8f2c506459e82a1d0cc34dccb605fddc214
child 207620 be9fedef6271629bb2a671ec8faf5604c57c2c35
push id494
push userraliiev@mozilla.com
push dateMon, 25 Aug 2014 18:42:16 +0000
treeherdermozilla-release@a3cc3e46b571 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssstamm
bugs1011211
milestone32.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1011211 - Remove unneeded QI check in CSP redirect tests r=sstamm
content/base/test/csp/test_301_redirect.html
content/base/test/csp/test_302_redirect.html
content/base/test/csp/test_303_redirect.html
content/base/test/csp/test_307_redirect.html
--- a/content/base/test/csp/test_301_redirect.html
+++ b/content/base/test/csp/test_301_redirect.html
@@ -23,20 +23,16 @@ Test that CSP violation reports are not 
 // This is used to watch the redirect of the report POST get blocked
 function examiner() {
   SpecialPowers.addObserver(this, "csp-on-violate-policy", false);
   SpecialPowers.addObserver(this, "specialpowers-http-notify-request", false);
 }
 
 examiner.prototype  = {
   observe: function(subject, topic, data) {
-    // subject should be an nsURI
-    if(!SpecialPowers.can_QI(subject))
-       return;
-
     if (topic === "specialpowers-http-notify-request") {
       // this is used to fail the test - if we see the POST to the target of the redirect
       // we know this is a fail
       var uri = data;
       if (uri == "http://example.com/some/fake/path")
         window.done(false);
     }
 
--- a/content/base/test/csp/test_302_redirect.html
+++ b/content/base/test/csp/test_302_redirect.html
@@ -23,20 +23,16 @@ Test that CSP violation reports are not 
 // This is used to watch the redirect of the report POST get blocked
 function examiner() {
   SpecialPowers.addObserver(this, "csp-on-violate-policy", false);
   SpecialPowers.addObserver(this, "specialpowers-http-notify-request", false);
 }
 
 examiner.prototype  = {
   observe: function(subject, topic, data) {
-    // subject should be an nsURI
-    if(!SpecialPowers.can_QI(subject))
-       return;
-
     if (topic === "specialpowers-http-notify-request") {
       // this is used to fail the test - if we see the POST to the target of the redirect
       // we know this is a fail
       var uri = data;
       if (uri == "http://example.com/some/fake/path")
         window.done(false);
     }
 
--- a/content/base/test/csp/test_303_redirect.html
+++ b/content/base/test/csp/test_303_redirect.html
@@ -23,20 +23,16 @@ Test that CSP violation reports are not 
 // This is used to watch the redirect of the report POST get blocked
 function examiner() {
   SpecialPowers.addObserver(this, "csp-on-violate-policy", false);
   SpecialPowers.addObserver(this, "specialpowers-http-notify-request", false);
 }
 
 examiner.prototype  = {
   observe: function(subject, topic, data) {
-    // subject should be an nsURI
-    if(!SpecialPowers.can_QI(subject))
-       return;
-
     if (topic === "specialpowers-http-notify-request") {
       // this is used to fail the test - if we see the POST to the target of the redirect
       // we know this is a fail
       var uri = data;
       if (uri == "http://example.com/some/fake/path")
         window.done(false);
     }
 
--- a/content/base/test/csp/test_307_redirect.html
+++ b/content/base/test/csp/test_307_redirect.html
@@ -23,20 +23,16 @@ Test that CSP violation reports are not 
 // This is used to watch the redirect of the report POST get blocked
 function examiner() {
   SpecialPowers.addObserver(this, "csp-on-violate-policy", false);
   SpecialPowers.addObserver(this, "specialpowers-http-notify-request", false);
 }
 
 examiner.prototype  = {
   observe: function(subject, topic, data) {
-    // subject should be an nsURI
-    if(!SpecialPowers.can_QI(subject))
-       return;
-
     if (topic === "specialpowers-http-notify-request") {
       // this is used to fail the test - if we see the POST to the target of the redirect
       // we know this is a fail
       var uri = data;
       if (uri == "http://example.com/some/fake/path")
         window.done(false);
     }