Cast:Updating logging stats

media/cast/logging/logging_stats.h

 // 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.
 
 #ifndef MEDIA_CAST_LOGGING_LOGGING_STATS_H_
 #define MEDIA_CAST_LOGGING_LOGGING_STATS_H_
 
 #include "base/basictypes.h"
 #include "base/time/time.h"
 #include "media/cast/logging/logging_defines.h"
@@ skipping 29 matching lines @@
                          CastLoggingEvent event,
                          uint32 rtp_timestamp,
                          uint32 frame_id,
                          uint16 packet_id,
                          uint16 max_packet_id,
                          size_t size);
 
   void InsertGenericEvent(const base::TimeTicks& time_of_event,
                           CastLoggingEvent event, int value);
 
-  // Get log stats: some of the values, such as frame rate and bit rates are
-  // computed at the time of the call.
-  const FrameStatsMap* GetFrameStatsData(const base::TimeTicks& now);
+  const FrameStatsMap* GetFrameStatsData() const;

[imcheng, 2014/01/22 21:03:37]

Can you provide some documentation? (e.g. ownership of returned object, etc.)

[mikhal1, 2014/01/23 19:53:11]

Done.

 
-  const PacketStatsMap* GetPacketStatsData(const base::TimeTicks& now);
+  const PacketStatsMap* GetPacketStatsData() const;
 
-  const GenericStatsMap* GetGenericStatsData();
+  const GenericStatsMap* GetGenericStatsData() const;
 
  private:
   void InsertBaseFrameEvent(const base::TimeTicks& time_of_event,
                             CastLoggingEvent event,
                             uint32 frame_id,
                             uint32 rtp_timestamp);
 
   FrameStatsMap frame_stats_;
   PacketStatsMap packet_stats_;
   GenericStatsMap generic_stats_;
-  // Every event has an individual start time
-  base::TimeTicks start_time_[kNumOfLoggingEvents];
-  // Keep track of event counts.
-  int counts_[kNumOfLoggingEvents];
 
   DISALLOW_COPY_AND_ASSIGN(LoggingStats);
 };
 
 }  // namespace cast
 }  // namespace media
 
 #endif  // MEDIA_CAST_LOGGING_LOGGING_STATS_H_