Index: talk/app/webrtc/webrtcsession.cc |
diff --git a/talk/app/webrtc/webrtcsession.cc b/talk/app/webrtc/webrtcsession.cc |
index 7241abf3cda5cdfc8e2b6052daba4a3b4d8d25d2..7f71961682cc3a97dfb1905a889b26675eb4a530 100644 |
--- a/talk/app/webrtc/webrtcsession.cc |
+++ b/talk/app/webrtc/webrtcsession.cc |
@@ -2149,33 +2149,32 @@ |
} |
const std::string& srtp_cipher = stats.channel_stats[0].srtp_cipher; |
- uint16_t ssl_cipher = stats.channel_stats[0].ssl_cipher; |
- if (srtp_cipher.empty() && !ssl_cipher) { |
+ const std::string& ssl_cipher = stats.channel_stats[0].ssl_cipher; |
+ if (srtp_cipher.empty() && ssl_cipher.empty()) { |
return; |
} |
- PeerConnectionEnumCounterType srtp_counter_type; |
- PeerConnectionEnumCounterType ssl_counter_type; |
+ PeerConnectionMetricsName srtp_name; |
+ PeerConnectionMetricsName ssl_name; |
if (stats.transport_name == cricket::CN_AUDIO) { |
- srtp_counter_type = kEnumCounterAudioSrtpCipher; |
- ssl_counter_type = kEnumCounterAudioSslCipher; |
+ srtp_name = kAudioSrtpCipher; |
+ ssl_name = kAudioSslCipher; |
} else if (stats.transport_name == cricket::CN_VIDEO) { |
- srtp_counter_type = kEnumCounterVideoSrtpCipher; |
- ssl_counter_type = kEnumCounterVideoSslCipher; |
+ srtp_name = kVideoSrtpCipher; |
+ ssl_name = kVideoSslCipher; |
} else if (stats.transport_name == cricket::CN_DATA) { |
- srtp_counter_type = kEnumCounterDataSrtpCipher; |
- ssl_counter_type = kEnumCounterDataSslCipher; |
+ srtp_name = kDataSrtpCipher; |
+ ssl_name = kDataSslCipher; |
} else { |
RTC_NOTREACHED(); |
return; |
} |
if (!srtp_cipher.empty()) { |
- metrics_observer_->IncrementSparseEnumCounter( |
- srtp_counter_type, rtc::GetSrtpCryptoSuiteFromName(srtp_cipher)); |
- } |
- if (ssl_cipher) { |
- metrics_observer_->IncrementSparseEnumCounter(ssl_counter_type, ssl_cipher); |
+ metrics_observer_->AddHistogramSample(srtp_name, srtp_cipher); |
+ } |
+ if (!ssl_cipher.empty()) { |
+ metrics_observer_->AddHistogramSample(ssl_name, ssl_cipher); |
} |
} |