Cast:Updating logging stats

media/cast/logging/logging_impl.cc

Index: media/cast/logging/logging_impl.cc
diff --git a/media/cast/logging/logging_impl.cc b/media/cast/logging/logging_impl.cc
index 90af985ce679062f997d574c2c813233a8292449..84882ef6f3ba0871412950349fc6ba9468b4cde8 100644
--- a/media/cast/logging/logging_impl.cc
+++ b/media/cast/logging/logging_impl.cc
@@ -183,17 +183,17 @@ void LoggingImpl::InsertGenericUmaEvent(CastLoggingEvent event, int value) {
 }
 
 // should just get the entire class, would be much easier.
-FrameRawMap LoggingImpl::GetFrameRawData() {
+FrameRawMap LoggingImpl::GetFrameRawData() const {
   DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
   return raw_.GetFrameData();
 }
 
-PacketRawMap LoggingImpl::GetPacketRawData() {
+PacketRawMap LoggingImpl::GetPacketRawData() const {
   DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
   return raw_.GetPacketData();
 }
 
-GenericRawMap LoggingImpl::GetGenericRawData() {
+GenericRawMap LoggingImpl::GetGenericRawData() const {
   DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
   return raw_.GetGenericData();
 }
@@ -208,124 +208,19 @@ VideoRtcpRawMap LoggingImpl::GetVideoRtcpRawData() {
   return raw_.GetAndResetVideoRtcpData();
 }
 
-const FrameStatsMap* LoggingImpl::GetFrameStatsData(
-    const base::TimeTicks& now) {
+const FrameStatsMap* LoggingImpl::GetFrameStatsData() const {
   DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
-  // Get stats data.
-  const FrameStatsMap* stats = stats_.GetFrameStatsData(now);
-  if (config_.enable_uma_stats) {
-    FrameStatsMap::const_iterator it;
-    for (it = stats->begin(); it != stats->end(); ++it) {
-      // Check for an active event.
-      // The default frame event implies frame rate.
-      if (it->second->framerate_fps > 0) {
-        switch (it->first) {
-          case kAudioFrameReceived:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.AudioFrameReceived",
-                it->second->framerate_fps);
-            break;
-          case kAudioFrameCaptured:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.AudioFrameCaptured",
-                it->second->framerate_fps);
-            break;
-          case kAudioFrameEncoded:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.AudioFrameEncoded",
-                it->second->framerate_fps);
-            break;
-          case kVideoFrameCaptured:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoFrameCaptured",
-                it->second->framerate_fps);
-            break;
-          case kVideoFrameReceived:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoFrameReceived",
-                it->second->framerate_fps);
-            break;
-          case kVideoFrameSentToEncoder:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoFrameSentToEncoder",
-                it->second->framerate_fps);
-            break;
-          case kVideoFrameEncoded:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoFrameEncoded",
-                it->second->framerate_fps);
-            break;
-          case kVideoFrameDecoded:
-            UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoFrameDecoded",
-                it->second->framerate_fps);
-            break;
-          default:
-            // No-op
-            break;
-        }
-      } else {
-        // All active frame events trigger frame rate computation.
-        continue;
-      }
-      // Bit rate should only be provided following encoding for either audio
-      // or video.
-      if (it->first == kVideoFrameEncoded) {
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoBitrateKbps",
-            it->second->framerate_fps);
-      } else if (it->first == kAudioFrameEncoded) {
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.AudioBitrateKbps",
-            it->second->framerate_fps);
-      }
-      // Delay events.
-      if (it->first == kAudioPlayoutDelay) {
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.AudioPlayoutDelayAvg",
-            it->second->avg_delay_ms);
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.AudioPlayoutDelayMin",
-            it->second->min_delay_ms);
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.AudioPlayoutDelayMax",
-            it->second->max_delay_ms);
-      } else if (it->first == kVideoRenderDelay) {
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoPlayoutDelayAvg",
-            it->second->avg_delay_ms);
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoPlayoutDelayMin",
-            it->second->min_delay_ms);
-        UMA_HISTOGRAM_COUNTS("Cast.Stats.VideoPlayoutDelayMax",
-            it->second->max_delay_ms);
-      }
-    }
-  }
-  return stats;
+  return stats_.GetFrameStatsData();
 }
 
-const PacketStatsMap* LoggingImpl::GetPacketStatsData(
-    const base::TimeTicks& now) {
+const PacketStatsMap* LoggingImpl::GetPacketStatsData() const {
   DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
-  // Get stats data.
-  const PacketStatsMap* stats = stats_.GetPacketStatsData(now);
-  if (config_.enable_uma_stats) {
-    PacketStatsMap::const_iterator it;
-    for (it = stats->begin(); it != stats->end(); ++it) {
-      switch (it->first) {
-        case kPacketSentToPacer:
-          UMA_HISTOGRAM_COUNTS("Cast.Stats.PacketSentToPacer", it->second);
-          break;
-        case kPacketSentToNetwork:
-          UMA_HISTOGRAM_COUNTS("Cast.Stats.PacketSentToNetwork", it->second);
-          break;
-        case kPacketRetransmitted:
-          UMA_HISTOGRAM_COUNTS("Cast.Stats.PacketRetransmited", it->second);
-          break;
-        case kDuplicatePacketReceived:
-          UMA_HISTOGRAM_COUNTS("Cast.Stats.DuplicatePacketReceived",
-                               it->second);
-          break;
-        default:
-          // No-op.
-          break;
-      }
-    }
-  }
-  return stats;
+  return stats_.GetPacketStatsData();
 }
 
-const GenericStatsMap* LoggingImpl::GetGenericStatsData() {
+const GenericStatsMap* LoggingImpl::GetGenericStatsData() const {
   DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
-  // Get stats data.
-  const GenericStatsMap* stats = stats_.GetGenericStatsData();
-  return stats;
+  return stats_.GetGenericStatsData();
 }
 
 void LoggingImpl::ResetRaw() {