Allow wrapping a media::VideoFrame with a new view_rect.

media/base/video_frame.h

Index: media/base/video_frame.h
diff --git a/media/base/video_frame.h b/media/base/video_frame.h
index 0ff6f9856ead06c2eeaa6380549e3ac05b116156..15d62fc77acaed2648bc22e1a55c9a60215a27ec 100644
--- a/media/base/video_frame.h
+++ b/media/base/video_frame.h
@@ -153,6 +153,7 @@ class MEDIA_EXPORT VideoFrame : public base::RefCountedThreadSafe<VideoFrame> {
   // gets destroyed.
   static scoped_refptr<VideoFrame> WrapVideoFrame(
       const scoped_refptr<VideoFrame>& frame,
+      const gfx::Rect& visible_rect,

[Ami GONE FROM CHROMIUM, 2014/03/05 16:23:42]

the impl makes me think the doco for this function wants to include:
"|visible_rect| must be a sub-rect of frame->visible_rect()."

Then the impl should actually enforce that (DCHECK).

[perkj_chrome, 2014/03/05 16:39:49]

sub size of of codec_size() right? 

I DCHECK(IsValidConfig) which seem to validate that and more.

       const base::Closure& no_longer_needed_cb);
 
   // Creates a frame which indicates end-of-stream.