admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove not-updating-stats log message.

Browse Source 
GetStats() is currently the only way to get smoothed audio levels. It's
expected to be called quite frequently. We use 100ms intervals, so a
call to GetStats() for any reason other than audio levels has a 50%
chance of triggering this log line. This makes it too noisy for LS_INFO.

The log line was added recently
(https://webrtc-review.googlesource.com/c/src/+/82260) and doesn't seem
very useful for diagnostic purposes, so remove it entirely.

Bug: webrtc:9519
Change-Id: I15700085c60b9929a4df2e2327012a4f16b505b6
Reviewed-on: https://webrtc-review.googlesource.com/88003
Commit-Queue: Jonathan Yu <yujo@chromium.org>
Reviewed-by: Noah Richards <noahric@chromium.org>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Honghai Zhang <honghaiz@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23939}
This commit is contained in:
Jonathan Yu 2018-07-10 12:45:34 -07:00 committed by Commit Bot
parent 78fef76e6a
commit 70aa374d3e
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pc/statscollector.cc
View File

@ -555,9 +555,6 @@ void StatsCollector::UpdateStats(
const double kMinGatherStatsPeriod = 50;
if (stats_gathering_started_ != 0 &&
stats_gathering_started_ + kMinGatherStatsPeriod > time_now) {
RTC_LOG(LS_INFO)
<< "Not updating stats again, since they were updated within "
<< kMinGatherStatsPeriod << "ms.";
return;
}
stats_gathering_started_ = time_now;