Do FinishAllRendering in WM_WINDOWPOSCHANGING

Do FinishAllRendering in WM_WINDOWPOSCHANGING

This ensures that the last swap completes before the window is resized, so that the window contents aren't scaled.

BUG=363704
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270206

[cpu_(ooo_6.6-7.5), 2014-05-10 02:36:23]

lgtm

adding iyengar for the hwmd message handler file.

https://codereview.chromium.org/277053003/diff/1/ui/compositor/compositor.cc
File ui/compositor/compositor.cc (left):

https://codereview.chromium.org/277053003/diff/1/ui/compositor/compositor.cc#...
ui/compositor/compositor.cc:293: // in the non-client area. See
RenderWidgetHostViewAura::SetBounds for
I might have put this code here :(

Do you know why is busted?

[jbauman, 2014-05-10 02:57:50]

On 2014/05/10 02:36:23, cpu wrote:
> lgtm
> 
> adding iyengar for the hwmd message handler file.
> 
> https://codereview.chromium.org/277053003/diff/1/ui/compositor/compositor.cc
> File ui/compositor/compositor.cc (left):
> 
>
https://codereview.chromium.org/277053003/diff/1/ui/compositor/compositor.cc#...
> ui/compositor/compositor.cc:293: // in the non-client area. See
> RenderWidgetHostViewAura::SetBounds for
> I might have put this code here :(
> 
> Do you know why is busted?

Yeah, the browser often draws frames that aren't caused by a resize (generally
because it got a frame from the renderer) in between resizes, and those won't be
waited for and can execute on the GPU process after the UI thread started
processing the next resize.

[jbauman, 2014-05-12 23:58:27]

ben@, could I get an OWNERS review for this?

[Ben Goodger (Google), 2014-05-13 19:56:08]

lgtm

[jbauman, 2014-05-13 20:07:10]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2014-05-13 20:09:35]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jbauman@chromium.org/277053003/1

[commit-bot: I haz the power, 2014-05-13 21:27:52]

Change committed as 270206