Cast: IPC from browser to renderer to send packet events from transport to cast library.

media/cast/audio_sender/audio_sender_unittest.cc

Index: media/cast/audio_sender/audio_sender_unittest.cc
diff --git a/media/cast/audio_sender/audio_sender_unittest.cc b/media/cast/audio_sender/audio_sender_unittest.cc
index dc2c3bf8ec6b61d0a7f0240deffb64af989fcc59..b756409e10628014aca16d0f97ef74ddca5a2ec6 100644
--- a/media/cast/audio_sender/audio_sender_unittest.cc
+++ b/media/cast/audio_sender/audio_sender_unittest.cc
@@ -84,6 +84,7 @@ class AudioSenderTest : public ::testing::Test {
         testing_clock_,
         transport_config,
         base::Bind(&UpdateCastTransportStatus),
+        base::Bind(&IgnoreRawEvents),
         task_runner_,
         &transport_));
     audio_sender_.reset(new AudioSender(
@@ -97,6 +98,10 @@ class AudioSenderTest : public ::testing::Test {
     EXPECT_EQ(status, transport::TRANSPORT_INITIALIZED);
   }
 
+  static void IgnoreRawEvents(const std::vector<PacketEvent>& packet_events) {
+    // Do nothing, we don't care about logging in this test.

[hubbe, 2014/02/26 01:01:07]

Is there *anything* we can add here to make it actually care a little bit?

[imcheng, 2014/02/26 10:08:26]

Not really. Logging is disabled in this test, so this callback is never invoked.
We could FAIL() here, but that's not very meaningful. This makes me wonder if
this could be designed a little bit better - from the caller's perspective, it
will be good to not have to specify callback if logging is disabled. Since only
some test code doesn't care about logging, I modified the constructor to check
for null callbacks, which it will interpret as "don't care about logging".

+  }
+
   base::SimpleTestTickClock* testing_clock_;  // Owned by CastEnvironment.
   TestPacketSender transport_;
   scoped_ptr<transport::CastTransportSenderImpl> transport_sender_;