Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(94)

Issue 1801993002: Strip content-encoding response header from CronetHttpURLConnection response. (Closed)

Created:
4 years, 9 months ago by mef
Modified:
4 years, 9 months ago
Reviewers:
xunjieli
CC:
chromium-reviews, cbentzel+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@2661
Target Ref:
refs/pending/branch-heads/2661
Project:
chromium
Visibility:
Public.

Description

Strip content-encoding response header from CronetHttpURLConnection response. Cronet decompresses response body before returning it to consumers, we should strip Content-Encoding headers so that consumers wouldn't try to decompress for a second time. BUG=592700 Review URL: https://codereview.chromium.org/1779473002 Cr-Commit-Position: refs/heads/master@{#379903} (cherry picked from commit cee2840c4bcd51313e33f0ab215413f9a8f70480) R=xunjieli@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/d25d5a281c748d5f9375972ecddd4306082386ff

Patch Set 1 #

Messages

Total messages: 5 (2 generated)
mef
4 years, 9 months ago (2016-03-14 19:52:25 UTC) #2
xunjieli
assuming there isn't any additional changes. lgtm
4 years, 9 months ago (2016-03-14 19:53:32 UTC) #3
mef
4 years, 9 months ago (2016-03-14 20:03:59 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
d25d5a281c748d5f9375972ecddd4306082386ff (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698