Commit cdc8420c authored by Nick Harper's avatar Nick Harper Committed by Commit Bot

Don't skip recording QUIC histograms

Change-Id: Ia91ea60c9dc239cac598153f422911ac84ca7cab
Bug: b/157458681
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320070
Auto-Submit: Nick Harper <nharper@chromium.org>
Commit-Queue: David Schinazi <dschinazi@chromium.org>
Reviewed-by: default avatarDavid Schinazi <dschinazi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#791979}
parent 89217066
...@@ -976,11 +976,6 @@ QuicChromiumClientSession::~QuicChromiumClientSession() { ...@@ -976,11 +976,6 @@ QuicChromiumClientSession::~QuicChromiumClientSession() {
// Sending one client_hello means we had zero handshake-round-trips. // Sending one client_hello means we had zero handshake-round-trips.
int round_trip_handshakes = crypto_stream_->num_sent_client_hellos() - 1; int round_trip_handshakes = crypto_stream_->num_sent_client_hellos() - 1;
// Don't bother with these histogram during tests, which mock out
// num_sent_client_hellos().
if (round_trip_handshakes < 0 || !stream_factory_)
return;
SSLInfo ssl_info; SSLInfo ssl_info;
// QUIC supports only secure urls. // QUIC supports only secure urls.
if (GetSSLInfo(&ssl_info) && ssl_info.cert.get()) { if (GetSSLInfo(&ssl_info) && ssl_info.cert.get()) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment