Chromium Code Reviews
Help | Chromium Project | Sign in
(277)

Issue 14273007: Add a switch to Chromium to enable/disable 'image/webp' accept header. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year ago by urvang (Google)
Modified:
1 year ago
Reviewers:
darin
CC:
chromium-reviews_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Add a switch to Chromium to enable/disable 'image/webp' accept header.

This switch is off by default. It will be turned on later once we verify that this change doesn't break any sites.

It can be can be enabled through command-line (--enable-webp-in-accept-header) or through 'chrome://flags'.

Note: This is a follow up of the pre-requisite change in Blink: https://codereview.chromium.org/13814024/

BUG=169182
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194853

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -1 line) Lint Patch
M chrome/app/generated_resources.grd View 1 chunk +6 lines, -0 lines 0 comments ? errors Download
M chrome/browser/about_flags.cc View 1 chunk +7 lines, -0 lines 0 comments 1 errors Download
M content/browser/renderer_host/render_process_host_impl.cc View 2 chunks +2 lines, -1 line 0 comments 0 errors Download
M content/public/common/content_switches.h View 1 chunk +2 lines, -0 lines 0 comments 0 errors Download
M content/public/common/content_switches.cc View 1 chunk +3 lines, -0 lines 0 comments 0 errors Download
M content/renderer/render_thread_impl.cc View 1 chunk +3 lines, -0 lines 0 comments 0 errors Download
Commit:

Messages

Total messages: 5
urvang (Google)
1 year ago #1
darin
LGTM
1 year ago #2
urvang (Google)
On 2013/04/16 04:29:39, darin wrote: > LGTM Thanks Darin! Let me know if you want ...
1 year ago #3
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/urvang@google.com/14273007/1
1 year ago #4
I haz the power (commit-bot)
1 year ago #5
Message was sent while issue was closed.
Change committed as 194853
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6