Add ability to copy unscaled output pixels.

Add ability to copy unscaled output pixels.

BUG=427418
Committed: https://crrev.com/fd59a531f36586b651f5550c66027d845092a9cf
Cr-Commit-Position: refs/heads/master@{#309542}

[eustas, 2014-12-15 13:12:44]

eustas@chromium.org changed reviewers:
+ dgozman@chromium.org

[eustas, 2014-12-15 13:39:08]

eustas@chromium.org changed reviewers:
+ jbauman@chromium.org

[eustas, 2014-12-15 13:39:09]

jbauman@chromium.org: Please review changes in compositor/

[eustas, 2014-12-15 13:40:01]

eustas@chromium.org changed reviewers:
+ asvitkine@chromium.org

[eustas, 2014-12-15 13:40:01]

asvitkine@chromium.org: Please review changes in renderer_host/

[dgozman, 2014-12-15 16:49:58]

Implementation looks fine, but you should update comment
render_widget_host_view_base.h.

[eustas, 2014-12-15 17:17:34]

On 2014/12/15 16:49:58, dgozman wrote:
> Implementation looks fine, but you should update comment
> render_widget_host_view_base.h.

Done.

[eustas, 2014-12-16 05:53:28]

eustas@chromium.org changed reviewers:
- asvitkine@chromium.org

[eustas, 2014-12-16 05:53:58]

eustas@chromium.org changed reviewers:
+ ccameron@chromium.org

[jbauman, 2014-12-16 05:55:42]

lgtm

[eustas, 2014-12-17 21:18:22]

eustas@chromium.org changed reviewers:
+ danakj@chromium.org
- ccameron@chromium.org, dgozman@chromium.org

[eustas, 2014-12-17 21:18:56]

eustas@chromium.org changed reviewers:
+ sievers@chromium.org
- danakj@chromium.org

[eustas, 2014-12-18 14:22:46]

Hello, Daniel.
Could you take a look at renderer_host/, please.
Thank you.

[no sievers, 2014-12-18 19:23:21]

https://codereview.chromium.org/802343002/diff/20001/content/browser/renderer...
File content/browser/renderer_host/render_widget_host_view_android.cc (right):

https://codereview.chromium.org/802343002/diff/20001/content/browser/renderer...
content/browser/renderer_host/render_widget_host_view_android.cc:1288:
output_size_in_pixel = src_subrect_in_pixel.size();
So we don't handle |src_subrect_in_pixel| being empty here?

[eustas, 2014-12-18 19:59:17]

https://codereview.chromium.org/802343002/diff/20001/content/browser/renderer...
File content/browser/renderer_host/render_widget_host_view_android.cc (right):

https://codereview.chromium.org/802343002/diff/20001/content/browser/renderer...
content/browser/renderer_host/render_widget_host_view_android.cc:1288:
output_size_in_pixel = src_subrect_in_pixel.size();
On 2014/12/18 19:23:20, sievers wrote:
> So we don't handle |src_subrect_in_pixel| being empty here?

Ooops! I'll take a closer look at this case.
Thank you.

[eustas, 2014-12-19 09:51:48]

PTAL

[eustas, 2014-12-22 08:31:02]

Ping

[no sievers, 2014-12-22 18:18:22]

lgtm

[eustas, 2014-12-23 05:33:38]

The CQ bit was checked by eustas@chromium.org

[commit-bot: I haz the power, 2014-12-23 05:34:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/802343002/40001

[commit-bot: I haz the power, 2014-12-23 07:13:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-12-23 07:14:20]

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