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

Unified Diff: net/http/http_server_properties.h

Issue 1572753003: QUIC - Allow cronet apps to specify how many server configs are to be (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: reabse TOT Created 4 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/http_server_properties.h
diff --git a/net/http/http_server_properties.h b/net/http/http_server_properties.h
index 9db9fc82888df8e45363acf79213aa4a01931e69..a9ce82b9afd6e0b014b772e7f87a6389677dd492 100644
--- a/net/http/http_server_properties.h
+++ b/net/http/http_server_properties.h
@@ -216,7 +216,7 @@ typedef base::MRUCache<HostPortPair, ServerNetworkStats> ServerNetworkStatsMap;
typedef base::MRUCache<QuicServerId, std::string> QuicServerInfoMap;
// Persist 5 QUIC Servers. This is mainly used by cronet.
-const int kMaxQuicServersToPersist = 5;
+const size_t kMaxQuicServersToPersist = 5;
mef 2016/01/15 16:12:32 should this remain int?
ramant (doing other things) 2016/01/15 18:57:00 Used size_t for the size everywhere to be consiste
extern const char kAlternateProtocolHeader[];
extern const char kAlternativeServiceHeader[];
@@ -380,6 +380,13 @@ class NET_EXPORT HttpServerProperties {
// Returns all persistent QuicServerInfo objects.
virtual const QuicServerInfoMap& quic_server_info_map() const = 0;
+ // Returns the number of server configs (QuicServerInfo objects) persisted.
+ virtual int max_server_configs_stored_in_properties() const = 0;
+
+ // Sets the number of server configs (QuicServerInfo objects) to be persisted.
+ virtual void set_max_server_configs_stored_in_properties(
+ int max_server_configs_stored_in_properties) = 0;
+
private:
DISALLOW_COPY_AND_ASSIGN(HttpServerProperties);
};

Powered by Google App Engine
This is Rietveld 408576698