Comment #13 on issue 13606 by [email protected]: Google search not  
working
http://code.google.com/p/chromium/issues/detail?id=13606

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=19925

------------------------------------------------------------------------
r19925 | [email protected] | 2009-07-03 18:11:37 -0700 (Fri, 03 Jul 2009) |  
23 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/net/base/filter.cc?r1=19925&r2=19924
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/net/base/gzip_filter_unittest.cc?r1=19925&r2=19924
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/net/base/sdch_filter.cc?r1=19925&r2=19924
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/net/base/sdch_filter_unittest.cc?r1=19925&r2=19924
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/net/base/sdch_manager.cc?r1=19925&r2=19924
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/net/base/sdch_manager.h?r1=19925&r2=19924

Merge r18936.

""  ==> "Tent_sdch, Tent_gzip"
"gzip" ==> "Tent_sdch, Test_gzip, gzip"

Note that the TENTATIVE_gzip is a sniffing gzip, that can turn into an
identity (no-op) filter if it can't find a gzip header.

By continuing to use a tentative SDCH, it is possible for the SDCH
filter to make error recovery decision, and to record stats
on this case.

In addition, I've changed the error recovery approach in the SDCH
filter to be ultra conservative.  If there is ANY unrecoverable
error in a response to an SDCH request, then we will blacklist
the host of the URL.  Recoverable errors are where the SDCH is fully
functional, and decodes its data using the dictionary.

The test cases added focus on the double gzip cases, as observed in
Vodaphone (UK) Mobile Broadband.

BUG=13606
Review URL: http://codereview.chromium.org/155064
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to