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

Issue 56034: Facilitate SDCH content decoding after a proxy removes ALL content info... (Closed)

Created:
11 years, 9 months ago by jar (doing other things)
Modified:
9 years, 7 months ago
Reviewers:
huanr, Lincoln
CC:
chromium-reviews_googlegroups.com, kmixter
Visibility:
Public.

Description

Facilitate SDCH content decoding after a proxy removes ALL content info Some proxies remove all content-encoding information when they see that SDCH encoding was done. We had a premature optimization that only called for doing "fixups" of content encoding IF there was some content encoding. We need to do the fixups as needed even if the HTTP headers didn't supply any encodings (such as when a proxy or anti-virus software strips all encodings!) bug=9151 r=huanr,kmixter,openvcdiff Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=12731

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -3 lines) Patch
M net/url_request/url_request_http_job.cc View 1 1 chunk +6 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jar (doing other things)
11 years, 9 months ago (2009-03-27 23:31:21 UTC) #1
huanr
lgtm
11 years, 9 months ago (2009-03-27 23:41:08 UTC) #2
Lincoln
11 years, 9 months ago (2009-03-27 23:44:25 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698