New issue
Advanced search Search tips

Issue 651821 link

Starred by 3 users
Status: WontFix
Owner: ----
Closed: Sep 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Get rid of SDCH meta refresh hack

Project Member Reported by xunji...@chromium.org, Sep 30 2016 
If a server sends a non-SDCH response with a Content-Encoding: sdch, URLRequestHttpJob issues a meta refresh to reload the resource without sdch support. This is a hack. We should investigate how to get rid of it.

Comment 1 by xunji...@chromium.org, Oct 10 2016 

This is in response to a review comment in https://codereview.chromium.org/2368433002/diff/140001/net/filter/sdch_policy_delegate.cc#newcode22.

When we reach SdchPolicyDelegate::OnDecodingError(), we have confirmed that there is a valid dictionary id and have successfully fetched the dictionary. We should really fail the request at this point rather than doing a meta refresh.

Comment 2 by xunji...@chromium.org, Oct 10 2016

Components: Internals>Network>SDCH

Comment 3 by xunji...@chromium.org, Sep 6 2017

Status: WontFix (was: Available)
SDCH has been disabled in Chrome since M59. For more details please see   Issue 690070   and blink-dev@ thread https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/nQl0ORHy7sw/HNpR96sqAgAJ.
►

Sign in to add a comment