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

Unified Diff: webrtc/pc/statscollector.cc

Issue 3011623002: Add new ANA stats to GetStats() to count the number of actions taken by each controller. (Closed)
Patch Set: Addressed review comments. Created 3 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/pc/statscollector.cc
diff --git a/webrtc/pc/statscollector.cc b/webrtc/pc/statscollector.cc
index 97bae3d189408b0bf3201d375f442ce8595191f9..db6fe3a0f3b8656d95342661995fdb1e1e08e6ce 100644
--- a/webrtc/pc/statscollector.cc
+++ b/webrtc/pc/statscollector.cc
@@ -224,6 +224,26 @@ void ExtractStats(const cricket::VoiceSenderInfo& info, StatsReport* report) {
}
}
report->AddString(StatsReport::kStatsValueNameMediaType, "audio");
+ if (info.ana_bitrate_action_counter) {
+ report->AddInt(StatsReport::kStatsValueNameAnaBitrateActionCounter,
+ *info.ana_bitrate_action_counter);
+ }
+ if (info.ana_channel_action_counter) {
+ report->AddInt(StatsReport::kStatsValueNameAnaChannelActionCounter,
+ *info.ana_channel_action_counter);
+ }
+ if (info.ana_dtx_action_counter) {
+ report->AddInt(StatsReport::kStatsValueNameAnaDtxActionCounter,
+ *info.ana_dtx_action_counter);
+ }
+ if (info.ana_fec_action_counter) {
+ report->AddInt(StatsReport::kStatsValueNameAnaFecActionCounter,
+ *info.ana_fec_action_counter);
+ }
+ if (info.ana_frame_length_action_counter) {
+ report->AddInt(StatsReport::kStatsValueNameAnaFrameLengthActionCounter,
+ *info.ana_frame_length_action_counter);
+ }
hbos 2017/09/04 08:49:54 Can you add a statscollector_unittest.cc to cover
ivoc 2017/09/04 15:48:06 Done.
}
void ExtractStats(const cricket::VideoReceiverInfo& info, StatsReport* report) {

Powered by Google App Engine
This is Rietveld 408576698