[Pkg-mozext-commits] [requestpolicy] 121/257: [ref] header redirect: don't remove header

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository requestpolicy.

commit 1ecff457e5cc4a47ef83b955a4af263dfd91b8fe
Author: Martin Kimmerle <dev at 256k.de>
Date:   Thu Oct 29 23:15:21 2015 +0100

    [ref] header redirect: don't remove header
    
    It's not necessary to remove the HTTP response header, because
    the request is canceled anyway.
---
 src/content/lib/request-processor.redirects.js | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/content/lib/request-processor.redirects.js b/src/content/lib/request-processor.redirects.js
index 78d8deb..1bfd01b 100644
--- a/src/content/lib/request-processor.redirects.js
+++ b/src/content/lib/request-processor.redirects.js
@@ -223,7 +223,9 @@ RequestProcessor = (function(self) {
     // The header isn't allowed, so remove it.
     try {
       if (!Prefs.isBlockingDisabled()) {
-        httpResponse.removeResponseHeader();
+        // Cancel the request. As of Fx 37, this causes the location bar to
+        // show the URL of the previously displayed page.
+        httpChannel.cancel(Cr.NS_BINDING_ABORTED);
 
         let browser = request.browser;
 
@@ -241,10 +243,6 @@ RequestProcessor = (function(self) {
           browser.rpcontinued.blockedRedirects[originURI] = destURI;
         }
 
-        // Cancel the request. As of Fx 37, this causes the location bar to
-        // show the URL of the previously displayed page.
-        httpChannel.cancel(Cr.NS_BINDING_ABORTED);
-
         eventuallyShowRedirectNotification(request);
 
         // We try to trace the blocked redirect back to a link click or form

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/requestpolicy.git