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

media/base/video_frame.cc

 // Copyright (c) 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.
 
 #include "media/base/video_frame.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
@@ skipping 162 matching lines @@
   frame->data_[kYPlane] = y_data;
   frame->data_[kUPlane] = u_data;
   frame->data_[kVPlane] = v_data;
   frame->no_longer_needed_cb_ = no_longer_needed_cb;
   return frame;
 }
 
 // static
 scoped_refptr<VideoFrame> VideoFrame::WrapVideoFrame(
       const scoped_refptr<VideoFrame>& frame,
+      const gfx::Rect& visible_rect,
       const base::Closure& no_longer_needed_cb) {
+  DCHECK(visible_rect.x() >= frame->visible_rect().x() &&
+         visible_rect.y() >= frame->visible_rect().y() &&
+         visible_rect.right() <= frame->visible_rect().right() &&
+         visible_rect.bottom() <= frame->visible_rect().bottom());

[Ami GONE FROM CHROMIUM, 2014/03/05 20:05:13]

aka DCHECK(frame->visible_rect().Contains(visible_rect));

[perkj_chrome, 2014/03/06 07:40:14]

thanks. 

   scoped_refptr<VideoFrame> wrapped_frame(new VideoFrame(
-      frame->format(), frame->coded_size(), frame->visible_rect(),
+      frame->format(), frame->coded_size(), visible_rect,
       frame->natural_size(), frame->GetTimestamp(), frame->end_of_stream()));
 
   for (size_t i = 0; i < NumPlanes(frame->format()); ++i) {
     wrapped_frame->strides_[i] = frame->stride(i);
     wrapped_frame->data_[i] = frame->data(i);
   }
 
   wrapped_frame->no_longer_needed_cb_ = no_longer_needed_cb;
   return wrapped_frame;
 }
@@ skipping 304 matching lines @@
       break;
     for (int row = 0; row < rows(plane); ++row) {
       base::MD5Update(context, base::StringPiece(
           reinterpret_cast<char*>(data(plane) + stride(plane) * row),
           row_bytes(plane)));
     }
   }
 }
 
 }  // namespace media