Cast: Switching recevier to use media::VideoFrame

media/cast/video_receiver/video_decoder.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_VIDEO_RECEIVER_VIDEO_DECODER_H_
 #define MEDIA_CAST_VIDEO_RECEIVER_VIDEO_DECODER_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/non_thread_safe.h"
 #include "media/cast/cast_config.h"
 #include "media/cast/cast_receiver.h"
 
 namespace media {
+class VideoFrame;

[scherkus (not reviewing), 2013/11/27 22:21:54]

nit: you can move this fwd decl to line 20

[mikhal1, 2013/11/27 22:41:30]

Done.

+}
+
+namespace media {
 namespace cast {
 
 class Vp8Decoder;
 
 // This class is not thread safe; it's only called from the cast video decoder
 // thread.
 class VideoDecoder : public base::NonThreadSafe {

[scherkus (not reviewing), 2013/11/27 22:21:54]

this is beyond the scope of this CL ... but is it possible to reuse
VpxVideoDecoder?

[mikhal1, 2013/11/27 22:41:30]

As you say, not in the scope of this cl. Added a TODO.
On 2013/11/27 22:21:54, scherkus wrote:

  public:
   VideoDecoder(const VideoReceiverConfig& video_config,
                scoped_refptr<CastEnvironment> cast_environment);
   virtual ~VideoDecoder();
 
   // Decode a video frame. Decoded (raw) frame will be returned via the
   // provided callback
   bool DecodeVideoFrame(const EncodedVideoFrame* encoded_frame,
                         const base::TimeTicks render_time,
                         const VideoFrameDecodedCallback& frame_decoded_cb);
 
  private:
   VideoCodec codec_;
   scoped_ptr<Vp8Decoder> vp8_decoder_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoDecoder);
 };
 
 }  // namespace cast
 }  // namespace media
 
 #endif  // MEDIA_CAST_VIDEO_RECEIVER_VIDEO_DECODER_H_