cc: Keep a reference on the media::VideoFrame.

cc/layers/video_layer_impl.h

 // Copyright 2012 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 CC_LAYERS_VIDEO_LAYER_IMPL_H_
 #define CC_LAYERS_VIDEO_LAYER_IMPL_H_
 
 #include "base/callback.h"
 #include "base/synchronization/lock.h"
 #include "cc/base/cc_export.h"
@@ skipping 52 matching lines @@
   virtual const char* LayerTypeAsString() const OVERRIDE;
 
   void WillDrawInternal(ResourceProvider* resource_provider);
   bool SetupFramePlanes(ResourceProvider* resource_provider);
   void FreeFramePlanes(ResourceProvider* resource_provider);
   void FreeUnusedFramePlanes(ResourceProvider* resource_provider);
   size_t NumPlanes() const;
 
   scoped_refptr<VideoFrameProviderClientImpl> provider_client_impl_;
 
-  media::VideoFrame* frame_;
+  scoped_refptr<media::VideoFrame> frame_;
   media::VideoFrame::Format format_;

[scherkus (not reviewing), 2013/03/21 20:29:56]

perhaps something to tackle in a separate CL: it looks like this might be
duplicate information from frame_->format()

it looks like format_ is derived from frame_->format(), except for when
convert_yuv_ is true ... but then in that case I wonder if we could check for
convert_yuv_ instead

[danakj, 2013/03/21 20:31:20]

Ya, there's some ongoing work by sheu@ to get rid of convert_yuv_ entirely.

   bool convert_yuv_;
   ResourceProvider::ResourceId external_texture_resource_;
   scoped_ptr<media::SkCanvasVideoRenderer> video_renderer_;
 
   // Each index in this array corresponds to a plane in media::VideoFrame.
   FramePlane frame_planes_[media::VideoFrame::kMaxPlanes];
 
   DISALLOW_COPY_AND_ASSIGN(VideoLayerImpl);
 };
 
 }  // namespace cc
 
 #endif  // CC_LAYERS_VIDEO_LAYER_IMPL_H_