Support copying a partial rectangle region from the compositing surface on Win.

Support copying a partial rectangle region from the compositing surface on Win.

BUG=118571
TEST=Manual


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148889

[mazda, 2012-07-27 18:19:05]

Hi Alastair,

Could you review this change?

[apatrick_chromium, 2012-07-27 18:28:19]

LGTM. You need an owner for content/browser.

http://codereview.chromium.org/10829054/diff/6001/ui/surface/accelerated_surf...
File ui/surface/accelerated_surface_win.cc (right):

http://codereview.chromium.org/10829054/diff/6001/ui/surface/accelerated_surf...
ui/surface/accelerated_surface_win.cc:124: int GetResampleCount(const gfx::Rect
src_subrect,
Did you mean to make this const gfx::Rect& ?

[mazda, 2012-07-27 19:29:10]

Thank you for the review.

+jam, could you do an OWNERS review for content/browser/renderer_host?

http://codereview.chromium.org/10829054/diff/6001/ui/surface/accelerated_surf...
File ui/surface/accelerated_surface_win.cc (right):

http://codereview.chromium.org/10829054/diff/6001/ui/surface/accelerated_surf...
ui/surface/accelerated_surface_win.cc:124: int GetResampleCount(const gfx::Rect
src_subrect,
On 2012/07/27 18:28:19, apatrick_chromium wrote:
> Did you mean to make this const gfx::Rect& ?

Done. Thanks.

[jam, 2012-07-27 23:48:06]

lgtm

[commit-bot: I haz the power, 2012-07-28 00:30:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mazda@chromium.org/10829054/3003

[commit-bot: I haz the power, 2012-07-28 02:29:43]

Try job failure for 10829054-3003 (retry) on win_rel for step "net_unittests".
It's a second try, previously, step "net_unittests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-07-28 04:05:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mazda@chromium.org/10829054/3003

[commit-bot: I haz the power, 2012-07-28 06:40:57]

Change committed as 148889