tor-browser

commit e79a1917851aab7ea43d19102653ddb4accb9c45
parent 981fe3517406ea1708b2bbf3a1172f1f670e64ac
Author: Serban Stanca <sstanca@mozilla.com>
Date:   Wed, 15 Oct 2025 20:36:22 +0300

Revert "Bug 1994472: Strip Content-Encoding when saving a cache item that's a dictionary r=necko-reviewers,valentin" for causing xpcshell failures in test_ext_webRequest_filterResponseData.js.

This reverts commit bf600905d4f6c85ba67e232635cbe3864b5a3acf.

Diffstat:
M
netwerk/protocol/http/HttpBaseChannel.cpp
 | 
8
++++----

1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/netwerk/protocol/http/HttpBaseChannel.cpp b/netwerk/protocol/http/HttpBaseChannel.cpp
@@ -1563,8 +1563,8 @@ HttpBaseChannel::DoApplyContentConversions(nsIStreamListener* aNextListener,
       }
       if (from.EqualsLiteral("dcb") || from.EqualsLiteral("dcz")) {
         MOZ_ASSERT(XRE_IsParentProcess());
+        removeEncodings = true;
       }
-      removeEncodings = true;
       nextListener = converter;
     } else {
       if (val) {
@@ -1584,9 +1584,9 @@ HttpBaseChannel::DoApplyContentConversions(nsIStreamListener* aNextListener,
   // if we need to remove any.
   // This double compression of course is silly, but supported by the spec.
   if (removeEncodings) {
-    // if we have dcb or dcz or if this is Use-As-Dictionary, all
-    // content-encodings in the header should be removed as we're
-    // decompressing before the tee in the parent process
+    // if we have dcb or dcz, all content-encodings in the header should
+    // be removed as we're decompressing before the tee in the parent
+    // process
     LOG(("Changing Content-Encoding from '%s' to ''", contentEncoding.get()));
     // Can't use SetHeader; we need to overwrite the current value
     rv = mResponseHead->SetHeaderOverride(nsHttp::Content_Encoding, ""_ns);