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

Issue 2897813002: Allow data reduction proxy (DRP) warmup URL fetch to use cookies (Closed)

Created:
3 years, 7 months ago by tbansal1
Modified:
3 years, 7 months ago
Reviewers:
RyanSturm
CC:
chromium-reviews, tbansal+watch-data-reduction-proxy_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Allow data reduction proxy (DRP) warmup URL fetch to use cookies This allows the sessions setup for fetching the warmup URL to be reused for fetching the user initiated requests. Currently, with cookies disabled, the warmup URL is fetched over a connection that has privacy mode enabled. This prevents the connection to be reused for fetching the user initiated requests later. BUG=725119 Review-Url: https://codereview.chromium.org/2897813002 Cr-Commit-Position: refs/heads/master@{#473618} Committed: https://chromium.googlesource.com/chromium/src/+/3374eeb70e0834cffcf3a9723e8a67042ded551b

Patch Set 1 : ps #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M components/data_reduction_proxy/core/browser/data_reduction_proxy_config.cc View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 21 (17 generated)
tbansal1
ryansturm: ptal. Thanks.
3 years, 7 months ago (2017-05-22 16:38:50 UTC) #9
RyanSturm
lgtm
3 years, 7 months ago (2017-05-22 16:52:44 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2897813002/20001
3 years, 7 months ago (2017-05-22 17:49:43 UTC) #17
commit-bot: I haz the power
3 years, 7 months ago (2017-05-22 17:56:11 UTC) #21
Message was sent while issue was closed.
Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/3374eeb70e0834cffcf3a9723e8a...

Powered by Google App Engine
This is Rietveld 408576698