[Cast] Refactor/clean-up VideoReceiver to match AudioReceiver as closely as possible.

media/cast/cast_receiver_impl.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_CAST_RECEIVER_IMPL_H_
 #define MEDIA_CAST_CAST_RECEIVER_IMPL_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "media/cast/audio_receiver/audio_receiver.h"
@@ skipping 19 matching lines @@
 
   // All received RTP and RTCP packets for the call should be sent to this
   // PacketReceiver;
   virtual transport::PacketReceiverCallback packet_receiver() OVERRIDE;
 
   // Interface to get audio and video frames from the CastReceiver.
   virtual scoped_refptr<FrameReceiver> frame_receiver() OVERRIDE;
 
  private:
   void ReceivedPacket(scoped_ptr<Packet> packet);
-  void UpdateTargetDelay(base::TimeDelta target_delay_ms);

[hubbe, 2014/04/07 18:40:17]

Why are we deleting this?

[miu, 2014/04/08 00:59:41]

It's a dead code path (no caller).  I'm the owner (I think) of implementing this
later on, once Mikhail figures out how the dynamic target delay adjustment is
supposed to work.  So, I wanted to remove this until we actually know what we're
going to do.

 
   transport::PacedSender pacer_;
   AudioReceiver audio_receiver_;
   VideoReceiver video_receiver_;
   scoped_refptr<FrameReceiver> frame_receiver_;
   scoped_refptr<CastEnvironment> cast_environment_;
   const uint32 ssrc_of_audio_sender_;
   const uint32 ssrc_of_video_sender_;
 
   DISALLOW_COPY_AND_ASSIGN(CastReceiverImpl);
 };
 
 }  // namespace cast
 }  // namespace media
 
 #endif  // MEDIA_CAST_CAST_RECEIVER_IMPL_