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

Unified Diff: google_apis/gcm/engine/connection_factory_impl.h

Issue 205343003: [GCM] Add port 443 fallback logic and histograms (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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: google_apis/gcm/engine/connection_factory_impl.h
diff --git a/google_apis/gcm/engine/connection_factory_impl.h b/google_apis/gcm/engine/connection_factory_impl.h
index 967c6c1ae46d0fc820ca02e51a82f4f8b4a297bb..caa3dadc7db8e29c069d2c2b7537649c51495c21 100644
--- a/google_apis/gcm/engine/connection_factory_impl.h
+++ b/google_apis/gcm/engine/connection_factory_impl.h
@@ -32,7 +32,7 @@ class GCM_EXPORT ConnectionFactoryImpl :
public net::NetworkChangeNotifier::IPAddressObserver {
public:
ConnectionFactoryImpl(
- const GURL& mcs_endpoint,
+ const std::vector<GURL>& mcs_endpoints,
const net::BackoffEntry::Policy& backoff_policy,
scoped_refptr<net::HttpNetworkSession> network_session,
net::NetLog* net_log);
@@ -89,8 +89,10 @@ class GCM_EXPORT ConnectionFactoryImpl :
void CloseSocket();
- // The MCS endpoint to make connections to.
- const GURL mcs_endpoint_;
+ // The MCS endpoints to make connections to, sorted in order of priority.
+ const std::vector<GURL> mcs_endpoints_;
+ // Index to the endpoint for a which a connection should be attempted next.
jianli 2014/03/20 01:11:53 nit: for which
Nicolas Zea 2014/03/20 20:10:02 Done.
+ size_t current_endpoint_;
// The backoff policy to use.
const net::BackoffEntry::Policy backoff_policy_;
« no previous file with comments | « no previous file | google_apis/gcm/engine/connection_factory_impl.cc » ('j') | google_apis/gcm/engine/connection_factory_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698