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

Issue 155065: Merge r19173.... (Closed)

Created:
11 years, 5 months ago by Mark Larson
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Base URL:
svn://chrome-svn.corp.google.com/chrome/branches/172/src/
Visibility:
Public.

Description

Merge r19173. Start to disable bzip2 support. First step: Stop advertising support. Although it would be nice to just yank it all, it is good to do so slowly so that any cached data that might required bzip2 decoding will still decode and render. Maybe I'm being paranoid... but this seems like a nice first small step. I also disabled the tests that depend on this modified code, and linted the unittest file a bit. BUG=14801 TEST= Be sure you can still view http://ukcp09.defra.gov.uk/content/view/16/6/index.html which is (currently) bzip encoded. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19926

Patch Set 1 #

Patch Set 2 : '' #

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

Messages

Total messages: 2 (0 generated)
Mark Larson
11 years, 5 months ago (2009-07-04 01:50:19 UTC) #1
jar (doing other things)
11 years, 5 months ago (2009-07-04 06:22:33 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698