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

Issue 577123003: Remove primary data reduction proxy workaround added by crbug.com/388358. (Closed)

Created:
6 years, 3 months ago by bolian
Modified:
6 years, 3 months ago
Reviewers:
bengr
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Remove primary data reduction proxy workaround added by crbug.com/388358. BUG=388987 Committed: https://crrev.com/3e8588fa8fd144eb68acd3965f3dedbfcf9e7673 Cr-Commit-Position: refs/heads/master@{#295586}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -25 lines) Patch
M components/data_reduction_proxy/browser/data_reduction_proxy_params.cc View 2 chunks +0 lines, -20 lines 0 comments Download
M components/data_reduction_proxy/browser/data_reduction_proxy_params_unittest.cc View 1 chunk +3 lines, -5 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
bolian
Hello, This CL is ready for review. Thanks.
6 years, 3 months ago (2014-09-18 20:58:32 UTC) #2
bengr
lgtm
6 years, 3 months ago (2014-09-18 21:06:29 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/577123003/1
6 years, 3 months ago (2014-09-18 21:35:36 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1) as 5323e5516798ed23319d0112c192b04ebd94d3e6
6 years, 3 months ago (2014-09-18 22:48:51 UTC) #6
commit-bot: I haz the power
6 years, 3 months ago (2014-09-18 22:49:15 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/3e8588fa8fd144eb68acd3965f3dedbfcf9e7673
Cr-Commit-Position: refs/heads/master@{#295586}

Powered by Google App Engine
This is Rietveld 408576698