Cast:Updating logging stats

media/cast/logging/logging_defines.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/cast/logging/logging_defines.h"
 
 #include "base/logging.h"
 
 namespace media {
 namespace cast {
@@ skipping 70 matching lines @@
     case(kVideoPacketReceived):
       return "VideoPacketReceived";
     case(kDuplicatePacketReceived):
       return "DuplicatePacketReceived";
     default:
       NOTREACHED();
       return "";
   }
 }
 
 FrameEvent::FrameEvent() {}

[imcheng, 2014/01/23 21:17:18]

Seems like we need to initialize the POD member varibles here and below as well?

[mikhal1, 2014/01/23 23:02:17]

Done.

 FrameEvent::~FrameEvent() {}
 
 BasePacketInfo::BasePacketInfo() {}
 BasePacketInfo::~BasePacketInfo() {}
 
 PacketEvent::PacketEvent() {}
 PacketEvent::~PacketEvent() {}
 
 GenericEvent::GenericEvent() {}
 GenericEvent::~GenericEvent() {}
 
 ReceiverRtcpEvent::ReceiverRtcpEvent() {}
 ReceiverRtcpEvent::~ReceiverRtcpEvent() {}
 
 FrameLogStats::FrameLogStats()
-    : framerate_fps(0),
-      bitrate_kbps(0),
-      max_delay_ms(0),
-      min_delay_ms(0),
-      avg_delay_ms(0) {}
+    : event_counter(0),
+      sum_size(0) {}
 FrameLogStats::~FrameLogStats() {}
 
+PacketLogStats::PacketLogStats()
+    : event_counter(0),
+      sum_size(0) {}
+
+PacketLogStats::~PacketLogStats() {}
+
+GenericLogStats::GenericLogStats()
+    : event_counter(0),
+      sum(0),
+      sum_squared(0),
+      min(0),
+      max(0) {}
+
+GenericLogStats::~GenericLogStats() {}
 }  // namespace cast
 }  // namespace media