Restrict the number of cases where we force the compositor to do a synchronous paint to resizing an…

Restrict the number of cases where we force the compositor to do a synchronous paint to resizing and restore/maximize operations.

Forcing the compositor to do a synchronous paint in window size changed notifications sends over
a synchronous IPC to the GPU process which blocks the UI thread. We have been seeing cases where
Chrome seems to freeze while performing operations like typing in the omnibox, etc.

BUG=397633
R=sky,cpu

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

[sky, 2014-09-03 14:53:51]

LGTM

https://codereview.chromium.org/528173002/diff/1/ui/views/widget/desktop_aura...
File ui/views/widget/desktop_aura/desktop_window_tree_host_win.h (right):

https://codereview.chromium.org/528173002/diff/1/ui/views/widget/desktop_aura...
ui/views/widget/desktop_aura/desktop_window_tree_host_win.h:268: // via the
compositor. Cases include restoring/resizing/maximizing the
via -> paint via?

https://codereview.chromium.org/528173002/diff/1/ui/views/widget/desktop_aura...
ui/views/widget/desktop_aura/desktop_window_tree_host_win.h:273: bool
sizing_notification_received_;
sizing_notification_received_ -> in_sizing_loop_

[ananta, 2014-09-03 17:44:55]

https://codereview.chromium.org/528173002/diff/1/ui/views/widget/desktop_aura...
File ui/views/widget/desktop_aura/desktop_window_tree_host_win.h (right):

https://codereview.chromium.org/528173002/diff/1/ui/views/widget/desktop_aura...
ui/views/widget/desktop_aura/desktop_window_tree_host_win.h:268: // via the
compositor. Cases include restoring/resizing/maximizing the
On 2014/09/03 14:53:50, sky wrote:
> via -> paint via?

Done.

https://codereview.chromium.org/528173002/diff/1/ui/views/widget/desktop_aura...
ui/views/widget/desktop_aura/desktop_window_tree_host_win.h:273: bool
sizing_notification_received_;
On 2014/09/03 14:53:50, sky wrote:
> sizing_notification_received_ -> in_sizing_loop_

Done.

[ananta, 2014-09-03 18:41:42]

The CQ bit was checked by ananta@chromium.org

[commit-bot: I haz the power, 2014-09-03 18:44:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ananta@chromium.org/528173002/20001

[cpu_(ooo_6.6-7.5), 2014-09-03 22:02:38]

lgtm

[commit-bot: I haz the power, 2014-09-03 23:06:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-03 23:06:13]

Failed to commit the patch.

[ananta, 2014-09-03 23:07:03]

The CQ bit was checked by ananta@chromium.org

[commit-bot: I haz the power, 2014-09-03 23:23:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ananta@chromium.org/528173002/20001

[commit-bot: I haz the power, 2014-09-03 23:46:29]

Committed patchset #2 (id:20001) as efb4ed49e60257ac52002ebd080c69a3ab35c3ce

[commit-bot: I haz the power, 2014-09-10 03:28:37]

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