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

Side by Side Diff: webrtc/api/peerconnection.cc

Issue 2386783002: Add UMA metrics for ICE regathering reasons. (Closed)
Patch Set: Add const to the pooled_sessions method. Created 4 years, 2 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 | « webrtc/api/fakemetricsobserver.cc ('k') | webrtc/api/umametrics.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 /* 1 /*
2 * Copyright 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 1282 matching lines...) Expand 10 before | Expand all | Expand 10 after
1293 void PeerConnection::RegisterUMAObserver(UMAObserver* observer) { 1293 void PeerConnection::RegisterUMAObserver(UMAObserver* observer) {
1294 TRACE_EVENT0("webrtc", "PeerConnection::RegisterUmaObserver"); 1294 TRACE_EVENT0("webrtc", "PeerConnection::RegisterUmaObserver");
1295 uma_observer_ = observer; 1295 uma_observer_ = observer;
1296 1296
1297 if (session_) { 1297 if (session_) {
1298 session_->set_metrics_observer(uma_observer_); 1298 session_->set_metrics_observer(uma_observer_);
1299 } 1299 }
1300 1300
1301 // Send information about IPv4/IPv6 status. 1301 // Send information about IPv4/IPv6 status.
1302 if (uma_observer_ && port_allocator_) { 1302 if (uma_observer_ && port_allocator_) {
1303 port_allocator_->SetMetricsObserver(uma_observer_);
1303 if (port_allocator_->flags() & cricket::PORTALLOCATOR_ENABLE_IPV6) { 1304 if (port_allocator_->flags() & cricket::PORTALLOCATOR_ENABLE_IPV6) {
1304 uma_observer_->IncrementEnumCounter( 1305 uma_observer_->IncrementEnumCounter(
1305 kEnumCounterAddressFamily, kPeerConnection_IPv6, 1306 kEnumCounterAddressFamily, kPeerConnection_IPv6,
1306 kPeerConnectionAddressFamilyCounter_Max); 1307 kPeerConnectionAddressFamilyCounter_Max);
1307 } else { 1308 } else {
1308 uma_observer_->IncrementEnumCounter( 1309 uma_observer_->IncrementEnumCounter(
1309 kEnumCounterAddressFamily, kPeerConnection_IPv4, 1310 kEnumCounterAddressFamily, kPeerConnection_IPv4,
1310 kPeerConnectionAddressFamilyCounter_Max); 1311 kPeerConnectionAddressFamilyCounter_Max);
1311 } 1312 }
1312 } 1313 }
(...skipping 1029 matching lines...) Expand 10 before | Expand all | Expand 10 after
2342 2343
2343 bool PeerConnection::StartRtcEventLog_w(rtc::PlatformFile file, 2344 bool PeerConnection::StartRtcEventLog_w(rtc::PlatformFile file,
2344 int64_t max_size_bytes) { 2345 int64_t max_size_bytes) {
2345 return media_controller_->call_w()->StartEventLog(file, max_size_bytes); 2346 return media_controller_->call_w()->StartEventLog(file, max_size_bytes);
2346 } 2347 }
2347 2348
2348 void PeerConnection::StopRtcEventLog_w() { 2349 void PeerConnection::StopRtcEventLog_w() {
2349 media_controller_->call_w()->StopEventLog(); 2350 media_controller_->call_w()->StopEventLog();
2350 } 2351 }
2351 } // namespace webrtc 2352 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/api/fakemetricsobserver.cc ('k') | webrtc/api/umametrics.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698