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

Side by Side Diff: net/socket/ssl_client_socket_pool.cc

Issue 869393005: Perform ClientHello padding if the field trial is enabled (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@net_connection_error_uma
Patch Set: CR update Created 5 years, 10 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 unified diff | Download patch
« no previous file with comments | « net/socket/ssl_client_socket_openssl.cc ('k') | net/ssl/ssl_config.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/socket/ssl_client_socket_pool.h" 5 #include "net/socket/ssl_client_socket_pool.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/metrics/field_trial.h" 9 #include "base/metrics/field_trial.h"
10 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
(...skipping 531 matching lines...) Expand 10 before | Expand all | Expand 10 after
542 UMA_HISTOGRAM_CUSTOM_TIMES("Net.SSL_Connection_Latency_Google_" 542 UMA_HISTOGRAM_CUSTOM_TIMES("Net.SSL_Connection_Latency_Google_"
543 "Full_Handshake", 543 "Full_Handshake",
544 connect_duration, 544 connect_duration,
545 base::TimeDelta::FromMilliseconds(1), 545 base::TimeDelta::FromMilliseconds(1),
546 base::TimeDelta::FromMinutes(1), 546 base::TimeDelta::FromMinutes(1),
547 100); 547 100);
548 } 548 }
549 } 549 }
550 } 550 }
551 551
552 UMA_HISTOGRAM_SPARSE_SLOWLY("Net.SSL_Connection_Error", std::abs(result));
553 if (params_->ssl_config().fastradio_padding_eligible) {
554 UMA_HISTOGRAM_SPARSE_SLOWLY("Net.SSL_Connection_Error_FastRadioPadding",
555 std::abs(result));
556 }
557
552 if (result == OK || IsCertificateError(result)) { 558 if (result == OK || IsCertificateError(result)) {
553 SetSocket(ssl_socket_.Pass()); 559 SetSocket(ssl_socket_.Pass());
554 } else if (result == ERR_SSL_CLIENT_AUTH_CERT_NEEDED) { 560 } else if (result == ERR_SSL_CLIENT_AUTH_CERT_NEEDED) {
555 error_response_info_.cert_request_info = new SSLCertRequestInfo; 561 error_response_info_.cert_request_info = new SSLCertRequestInfo;
556 ssl_socket_->GetSSLCertRequestInfo( 562 ssl_socket_->GetSSLCertRequestInfo(
557 error_response_info_.cert_request_info.get()); 563 error_response_info_.cert_request_info.get());
558 } 564 }
559 565
560 return result; 566 return result;
561 } 567 }
(...skipping 277 matching lines...) Expand 10 before | Expand all | Expand 10 after
839 CHECK(it != messenger_map_.end()); 845 CHECK(it != messenger_map_.end());
840 delete it->second; 846 delete it->second;
841 messenger_map_.erase(it); 847 messenger_map_.erase(it);
842 } 848 }
843 849
844 void SSLClientSocketPool::OnSSLConfigChanged() { 850 void SSLClientSocketPool::OnSSLConfigChanged() {
845 FlushWithError(ERR_NETWORK_CHANGED); 851 FlushWithError(ERR_NETWORK_CHANGED);
846 } 852 }
847 853
848 } // namespace net 854 } // namespace net
OLDNEW
« no previous file with comments | « net/socket/ssl_client_socket_openssl.cc ('k') | net/ssl/ssl_config.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698