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

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

Issue 2315613002: Extracted NetLog class's inner enum types into their own enum classes and (Closed)
Patch Set: Ran "git cl format" on code. Much formatting ensued. Created 4 years, 3 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
« no previous file with comments | « extensions/browser/extension_throttle_manager.cc ('k') | ios/chrome/browser/ios_chrome_io_thread.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: google_apis/gcm/engine/connection_factory_impl.cc
diff --git a/google_apis/gcm/engine/connection_factory_impl.cc b/google_apis/gcm/engine/connection_factory_impl.cc
index ccfb9cd5e8305510f061ba38bc376db080c4921d..d0035b18153a7abf971d7e8dd195daa7a7837a9c 100644
--- a/google_apis/gcm/engine/connection_factory_impl.cc
+++ b/google_apis/gcm/engine/connection_factory_impl.cc
@@ -19,6 +19,7 @@
#include "net/http/http_network_session.h"
#include "net/http/http_request_headers.h"
#include "net/log/net_log.h"
+#include "net/log/net_log_source_type.h"
#include "net/proxy/proxy_info.h"
#include "net/socket/client_socket_handle.h"
#include "net/socket/client_socket_pool_manager.h"
@@ -54,22 +55,22 @@ ConnectionFactoryImpl::ConnectionFactoryImpl(
net::HttpNetworkSession* http_network_session,
net::NetLog* net_log,
GCMStatsRecorder* recorder)
- : mcs_endpoints_(mcs_endpoints),
- next_endpoint_(0),
- last_successful_endpoint_(0),
- backoff_policy_(backoff_policy),
- gcm_network_session_(gcm_network_session),
- http_network_session_(http_network_session),
- bound_net_log_(
- net::BoundNetLog::Make(net_log, net::NetLog::SOURCE_SOCKET)),
- pac_request_(NULL),
- connecting_(false),
- waiting_for_backoff_(false),
- waiting_for_network_online_(false),
- logging_in_(false),
- recorder_(recorder),
- listener_(NULL),
- weak_ptr_factory_(this) {
+ : mcs_endpoints_(mcs_endpoints),
+ next_endpoint_(0),
+ last_successful_endpoint_(0),
+ backoff_policy_(backoff_policy),
+ gcm_network_session_(gcm_network_session),
+ http_network_session_(http_network_session),
+ bound_net_log_(
+ net::BoundNetLog::Make(net_log, net::NetLogSourceType::SOCKET)),
+ pac_request_(NULL),
+ connecting_(false),
+ waiting_for_backoff_(false),
+ waiting_for_network_online_(false),
+ logging_in_(false),
+ recorder_(recorder),
+ listener_(NULL),
+ weak_ptr_factory_(this) {
DCHECK_GE(mcs_endpoints_.size(), 1U);
DCHECK(!http_network_session_ ||
(gcm_network_session_ != http_network_session_));
« no previous file with comments | « extensions/browser/extension_throttle_manager.cc ('k') | ios/chrome/browser/ios_chrome_io_thread.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698