Bug 1267033 - Ensure QI to nsIHttpChannel before trying to merge status info. r?kmag draft
authorGiorgio Maone <g.maone@informaction.com>
Wed, 27 Apr 2016 18:11:32 +0200
changeset 356916 aa59cade3cfa998b8d6a1f5ab14cee25ce20d252
parent 356894 ab0044bfa1df858919797bcd6a9aef76a668cd4a
child 519524 c6fad166e84a63953b537069ee2f31a966107f11
push id16645
push userg.maone@informaction.com
push dateWed, 27 Apr 2016 16:12:36 +0000
reviewerskmag
bugs1267033
milestone49.0a1
Bug 1267033 - Ensure QI to nsIHttpChannel before trying to merge status info. r?kmag MozReview-Commit-ID: 5kZndCaQmPr
toolkit/modules/addons/WebRequest.jsm
--- a/toolkit/modules/addons/WebRequest.jsm
+++ b/toolkit/modules/addons/WebRequest.jsm
@@ -500,20 +500,22 @@ HttpObserverManager = {
     let loadInfo = channel.loadInfo;
     let policyType = loadInfo ?
                      loadInfo.externalContentPolicyType :
                      Ci.nsIContentPolicy.TYPE_OTHER;
 
     let requestHeaderNames;
     let responseHeaderNames;
 
-    let includeStatus = kind === "headersReceived" ||
-                        kind === "onRedirect" ||
-                        kind === "onStart" ||
-                        kind === "onStop";
+    let includeStatus = (
+                          kind === "headersReceived" ||
+                          kind === "onRedirect" ||
+                          kind === "onStart" ||
+                          kind === "onStop"
+                        ) && channel instanceof Ci.nsIHttpChannel;
 
     let commonData = null;
     let uri = channel.URI;
     for (let [callback, opts] of listeners.entries()) {
       if (!this.shouldRunListener(policyType, uri, opts.filter)) {
         continue;
       }