Incorporating logging into Cast

media/cast/cast_sender_impl.cc

Index: media/cast/cast_sender_impl.cc
diff --git a/media/cast/cast_sender_impl.cc b/media/cast/cast_sender_impl.cc
index 1bdd7c51f28775b27796da1302e782d975acf93f..4e1107a541ccd0b0d2af2df20c83985f1f1b2bb2 100644
--- a/media/cast/cast_sender_impl.cc
+++ b/media/cast/cast_sender_impl.cc
@@ -34,6 +34,9 @@ class LocalFrameInput : public FrameInput {
   virtual void InsertRawVideoFrame(const I420VideoFrame* video_frame,
                                    const base::TimeTicks& capture_time,
                                    const base::Closure callback) OVERRIDE {
+  cast_environment_->Logging()->InsertFrameEvent(kVideoFrameReceived,
+      GetVideoRtpTimestamp(capture_time), -1);

[Alpha Left Google, 2013/11/14 00:29:24]

Add a TODO here to replace -1 with something meaningful.

[mikhal, 2013/11/14 17:42:31]

Added a const value for his. The -1 will be a valid entry once the frame_id are
represented in ints (pwestin pending cl).
On 2013/11/14 00:29:24, Alpha wrote:

+
     cast_environment_->PostTask(CastEnvironment::MAIN, FROM_HERE,
         base::Bind(&VideoSender::InsertRawVideoFrame, video_sender_,
             video_frame, capture_time, callback));
@@ -50,6 +53,8 @@ class LocalFrameInput : public FrameInput {
   virtual void InsertAudio(const AudioBus* audio_bus,
                            const base::TimeTicks& recorded_time,
                            const base::Closure& done_callback) OVERRIDE {
+    cast_environment_->Logging()->InsertFrameEvent(kAudioFrameReceived,
+        GetVideoRtpTimestamp(recorded_time), -1);

[Alpha Left Google, 2013/11/14 00:29:24]

Add a TODO here to replace -1 with something meaningful.

[mikhal, 2013/11/14 17:42:31]

Same as above
On 2013/11/14 00:29:24, Alpha wrote:

     cast_environment_->PostTask(CastEnvironment::MAIN, FROM_HERE,
         base::Bind(&AudioSender::InsertAudio, audio_sender_,
             audio_bus, recorded_time, done_callback));