Gather metrics classifying the cause of the TLS fallback.

net/socket/client_socket_handle.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/socket/client_socket_handle.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "base/metrics/histogram.h"
 #include "base/logging.h"
 #include "net/base/net_errors.h"
 #include "net/socket/client_socket_pool.h"
 
 namespace net {
 
 ClientSocketHandle::ClientSocketHandle()
     : is_initialized_(false),
       pool_(NULL),
       higher_pool_(NULL),
       reuse_type_(ClientSocketHandle::UNUSED),
       callback_(base::Bind(&ClientSocketHandle::OnIOComplete,
                            base::Unretained(this))),
-      is_ssl_error_(false) {}
+      is_ssl_error_(false),
+      ssl_failure_state_(kSSLFailureNone) {
+}
 
 ClientSocketHandle::~ClientSocketHandle() {
   Reset();
 }
 
 void ClientSocketHandle::Reset() {
   ResetInternal(true);
   ResetErrorState();
 }
 
@@ skipping 30 matching lines @@
   idle_time_ = base::TimeDelta();
   init_time_ = base::TimeTicks();
   setup_time_ = base::TimeDelta();
   connect_timing_ = LoadTimingInfo::ConnectTiming();
   pool_id_ = -1;
 }
 
 void ClientSocketHandle::ResetErrorState() {
   is_ssl_error_ = false;
   ssl_error_response_info_ = HttpResponseInfo();
   pending_http_proxy_connection_.reset();

[Ryan Sleevi, 2015/05/13 22:26:36]

BUG?: You forgot to reset the SSLFailureState here

[davidben, 2015/05/14 23:24:34]

Done. (Sigh. Why must we have so many manual "reset state" methods? I much
prefer a "create object anew if reseting universe.)

 }
 
 LoadState ClientSocketHandle::GetLoadState() const {
   CHECK(!is_initialized());
   CHECK(!group_name_.empty());
   // Because of http://crbug.com/37810  we may not have a pool, but have
   // just a raw socket.
   if (!pool_)
     return LOAD_STATE_IDLE;
   return pool_->GetLoadState(group_name_, this);
@@ skipping 77 matching lines @@
   // TODO(eroman): This logging is not complete, in particular set_socket() and
   // release() socket. It ends up working though, since those methods are being
   // used to layer sockets (and the destination sources are the same).
   DCHECK(socket_.get());
   socket_->NetLog().BeginEvent(
       NetLog::TYPE_SOCKET_IN_USE,
       requesting_source_.ToEventParametersCallback());
 }
 
 }  // namespace net