media: GMBVideoFramePool, crop VideoFrame using visible_rect.

media: GMBVideoFramePool, crop VideoFrame using visible_rect.

VideoFrame::visible_rect could be different than natural_size when the
VideoFrame is coming from a media stream that is from a camera.
crrev.com/1358883003 is planning to convert the video frames coming from
different media sources.

This CL crops VideoFrame using visible_rect when it gets converted to
a new VideoFrame backed by GpuMemoryBuffers.

BUG=

Committed: https://crrev.com/ab977781fdcd23b3548202e9e422cd30edb67067
Cr-Commit-Position: refs/heads/master@{#351149}

[Daniele Castagna, 2015-09-25 18:32:55]

dcastagna@chromium.org changed reviewers:
+ emircan@chromium.org

[emircan, 2015-09-26 02:20:12]

lgtm. Thanks.

https://codereview.chromium.org/1371683002/diff/1/media/video/gpu_memory_buff...
File media/video/gpu_memory_buffer_video_frame_pool.cc (right):

https://codereview.chromium.org/1371683002/diff/1/media/video/gpu_memory_buff...
media/video/gpu_memory_buffer_video_frame_pool.cc:393: gfx::Size size =
video_frame->visible_rect().size();
const

[Daniele Castagna, 2015-09-28 15:46:57]

dcastagna@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[Daniele Castagna, 2015-09-28 15:46:57]

+dalecurtis for ownership.

https://codereview.chromium.org/1371683002/diff/1/media/video/gpu_memory_buff...
File media/video/gpu_memory_buffer_video_frame_pool.cc (right):

https://codereview.chromium.org/1371683002/diff/1/media/video/gpu_memory_buff...
media/video/gpu_memory_buffer_video_frame_pool.cc:393: gfx::Size size =
video_frame->visible_rect().size();
On 2015/09/26 at 02:20:12, emircan wrote:
> const

Done.

[DaleCurtis, 2015-09-28 20:56:43]

lgtm

[Daniele Castagna, 2015-09-28 20:57:03]

The CQ bit was checked by dcastagna@chromium.org

[Daniele Castagna, 2015-09-28 20:57:04]

The patchset sent to the CQ was uploaded after l-g-t-m from emircan@chromium.org
Link to the patchset: https://codereview.chromium.org/1371683002/#ps20001
(title: "const size.")

[commit-bot: I haz the power, 2015-09-28 20:58:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1371683002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1371683002/20001

[commit-bot: I haz the power, 2015-09-28 21:08:01]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-28 21:11:20]

Patchset 2 (id:??) landed as
https://crrev.com/ab977781fdcd23b3548202e9e422cd30edb67067
Cr-Commit-Position: refs/heads/master@{#351149}