webrtc/video/call_stats.cc - Issue 1788783002: Add macros for ability to log samples that are added to histograms (RTC_LOGGED_*).

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: webrtc/video/call_stats.cc

Issue 1788783002: Add macros for ability to log samples that are added to histograms (RTC_LOGGED_*). (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master

Patch Set: Created 4 years, 9 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/system_wrappers/include/metrics.h ('k') | webrtc/video/receive_statistics_proxy.cc » ('j') | no next file with comments »

OLD	NEW
1 /*	1 /*

2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.	2 * Copyright (c) 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 163 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
174	174

175 void CallStats::UpdateHistograms() {	175 void CallStats::UpdateHistograms() {

176 rtc::CritScope cs(&crit_);	176 rtc::CritScope cs(&crit_);

177 if (time_of_first_rtt_ms_ == -1 \|\| num_avg_rtt_ < 1)	177 if (time_of_first_rtt_ms_ == -1 \|\| num_avg_rtt_ < 1)

178 return;	178 return;

179	179

180 int64_t elapsed_sec =	180 int64_t elapsed_sec =

181 (clock_->TimeInMilliseconds() - time_of_first_rtt_ms_) / 1000;	181 (clock_->TimeInMilliseconds() - time_of_first_rtt_ms_) / 1000;

182 if (elapsed_sec >= metrics::kMinRunTimeInSeconds) {	182 if (elapsed_sec >= metrics::kMinRunTimeInSeconds) {

183 int64_t avg_rtt_ms = (sum_avg_rtt_ms_ + num_avg_rtt_ / 2) / num_avg_rtt_;	183 int64_t avg_rtt_ms = (sum_avg_rtt_ms_ + num_avg_rtt_ / 2) / num_avg_rtt_;

184 RTC_HISTOGRAM_COUNTS_10000(	184 RTC_LOGGED_HISTOGRAM_COUNTS_10000(

185 "WebRTC.Video.AverageRoundTripTimeInMilliseconds", avg_rtt_ms);	185 "WebRTC.Video.AverageRoundTripTimeInMilliseconds", avg_rtt_ms);

186 }	186 }

187 }	187 }

188	188

189 } // namespace webrtc	189 } // namespace webrtc

OLD	NEW