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

Unified Diff: net/http/mock_http_cache.h

Issue 2668563002: Change MockHttpCache constructor to take boolean to set quic server info factory (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/http/mock_http_cache.h
diff --git a/net/http/mock_http_cache.h b/net/http/mock_http_cache.h
index 7cf03db4051170a08b6d579624f2669cdd623f73..17403f48f5f210bb3d9c2b1eb496f6ee1eac0618 100644
--- a/net/http/mock_http_cache.h
+++ b/net/http/mock_http_cache.h
@@ -179,9 +179,10 @@ class MockBackendFactory : public HttpCache::BackendFactory {
class MockHttpCache {
public:
- MockHttpCache();
+ MockHttpCache(bool set_up_quic_server_info);
Ryan Hamilton 2017/01/31 01:32:26 nit: can you comment what this param means? Also,
Zhongyi Shi 2017/02/02 01:05:12 Done.
explicit MockHttpCache(
Ryan Hamilton 2017/01/31 01:32:26 explicit is no longer needed.
Zhongyi Shi 2017/02/02 01:05:12 Done.
- std::unique_ptr<HttpCache::BackendFactory> disk_cache_factory);
+ std::unique_ptr<HttpCache::BackendFactory> disk_cache_factory,
+ bool set_up_quic_server_info);
HttpCache* http_cache() { return &http_cache_; }

Powered by Google App Engine
This is Rietveld 408576698