cast: Wire upp logging to be sent over RTCP between receiver and sender.

media/cast/logging/logging_raw.h

Index: media/cast/logging/logging_raw.h
diff --git a/media/cast/logging/logging_raw.h b/media/cast/logging/logging_raw.h
index 79111785e84fb336f8a814312581ed9a1b55581b..504a60c911f5ff4c7440c974bdd7498e64189e2f 100644
--- a/media/cast/logging/logging_raw.h
+++ b/media/cast/logging/logging_raw.h
@@ -24,7 +24,7 @@ namespace cast {
 class LoggingRaw : public base::NonThreadSafe,
                    public base::SupportsWeakPtr<LoggingRaw> {
  public:
-  LoggingRaw();
+  explicit LoggingRaw(bool is_sender);
   ~LoggingRaw();
 
   // Inform of new event: three types of events: frame, packets and generic.
@@ -66,8 +66,10 @@ class LoggingRaw : public base::NonThreadSafe,
   PacketRawMap GetPacketData() const;
   GenericRawMap GetGenericData() const;
 
+  AudioRtcpRawMap GetAndResetAudioRtcpData();
+  VideoRtcpRawMap GetAndResetVideoRtcpData();
 
-  // Reset all log data.
+  // Reset all log data; except the Rtcp copies.
   void Reset();
 
  private:
@@ -76,9 +78,18 @@ class LoggingRaw : public base::NonThreadSafe,
                             uint32 frame_id,
                             uint32 rtp_timestamp);
 
+  void InsertRtcpFrameEvent(const base::TimeTicks& time_of_event,

[Alpha Left Google, 2014/01/21 21:14:45]

I'm quite confused. What is the purpose of having two "events"? One for
BaseFrameEvent and RtcpFrameEvent? What are they used for and what are they? Can
you please add some comments.

[pwestin, 2014/01/21 22:00:28]

They are driven by very different timers and we also don't want to create a
infinite loop between sender and receiver. If the log class could handle
multiple listeners that run in different pace this could be avoided. 

+                            CastLoggingEvent event,
+                            uint32 rtp_timestamp,
+                            base::TimeDelta delay);
+
+  const bool is_sender_;
   FrameRawMap frame_map_;
   PacketRawMap packet_map_;
   GenericRawMap generic_map_;
+  AudioRtcpRawMap audio_rtcp_map_;
+  VideoRtcpRawMap video_rtcp_map_;
+
   base::WeakPtrFactory<LoggingRaw> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(LoggingRaw);