Cast: Refactor RTCP handling

chrome/renderer/media/cast_ipc_dispatcher_unittest.cc

Index: chrome/renderer/media/cast_ipc_dispatcher_unittest.cc
diff --git a/chrome/renderer/media/cast_ipc_dispatcher_unittest.cc b/chrome/renderer/media/cast_ipc_dispatcher_unittest.cc
index 1f330a172a8a2be1169ec8b1a2a74c64e6bf733c..1491e5617a4d76f131e72ad42f348f11524e77b4 100644
--- a/chrome/renderer/media/cast_ipc_dispatcher_unittest.cc
+++ b/chrome/renderer/media/cast_ipc_dispatcher_unittest.cc
@@ -39,11 +39,19 @@ TEST_F(CastIPCDispatcherTest, RawEvents) {
   packet_event.type = media::cast::PACKET_SENT_TO_NETWORK;
   packet_event.media_type = media::cast::VIDEO_EVENT;
 
+  std::vector<media::cast::FrameEvent> frame_events;
+  media::cast::FrameEvent frame_event;

[miu, 2014/07/18 00:06:23]

I think you meant this LOC to be these two:

  frame_events.push_back();
  media::cast::FrameEvent& frame_event = frame_events.back();

Otherwise, you're always sending an empty frame_events vector.  ;-)

[Alpha Left Google, 2014/07/18 01:14:30]

Done.

+  frame_event.rtp_timestamp = 100;
+  frame_event.frame_id = 5;
+  frame_event.size = 512;
+  frame_event.timestamp = base::SimpleTestTickClock().NowTicks();
+  frame_event.media_type = media::cast::VIDEO_EVENT;
+
   packet_events.push_back(packet_event);
-  CastMsg_RawEvents raw_events_msg(kChannelId, packet_events);
+  CastMsg_RawEvents raw_events_msg(kChannelId, packet_events,
+                                   frame_events);
 
   FakeSend(raw_events_msg);
 }
 
 }  // namespace
-