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

Issue 1038203002: QUIC - minor cleanup while merging the following change. (Closed)

Created:
5 years, 9 months ago by ramant (doing other things)
Modified:
5 years, 9 months ago
Reviewers:
Ryan Hamilton
CC:
chromium-reviews, cbentzel+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@rch_patch_1036023002
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

QUIC - minor cleanup while merging the following change. In Chromium's AddToCache function, we use "www.google.com" instead of empty path as it was done in the internal CL. All other changes were already merged by rch. Make some changes to the QuicInMemory server to clarify the interface. Specifically that the cache is indexed by host and path, not by method nor scheme, nor full URL, nor request headers, etc. In preparation for making the chrome version more similar. Merge internal change: 89092881 R=rch@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M net/tools/quic/quic_in_memory_cache.h View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
ramant (doing other things)
5 years, 9 months ago (2015-03-26 23:25:32 UTC) #1
Ryan Hamilton
5 years, 9 months ago (2015-03-27 03:02:46 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698