views: Fix a couple of flaky tests on windows/linux, disable them on mus.

views: Fix a couple of flaky tests on windows/linux, disable them on mus.

We can no longer expect a single cycle of message-loop iteration to
paint a widget, since painting is now driven by BeginFrameSource. So
update the tests to wait for the painting to happen first.

These tests create desktop-widgets, which we do not support in mus, so
disable the tests in mus.

BUG=596039
Committed: https://crrev.com/3845af20c5138df223fd0fba3eb5a275aacea6b8
Cr-Commit-Position: refs/heads/master@{#397206}

[sadrul, 2016-06-01 16:50:20]

sadrul@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2016-06-01 17:18:27]

LGTM

https://codereview.chromium.org/2032573002/diff/20001/ui/views/widget/widget_...
File ui/views/widget/widget_unittest.cc (right):

https://codereview.chromium.org/2032573002/diff/20001/ui/views/widget/widget_...
ui/views/widget/widget_unittest.cc:1402: if (IsMus())
Can you link this and the next to bugs with a TODO?

[sadrul, 2016-06-01 18:28:55]

https://codereview.chromium.org/2032573002/diff/20001/ui/views/widget/widget_...
File ui/views/widget/widget_unittest.cc (right):

https://codereview.chromium.org/2032573002/diff/20001/ui/views/widget/widget_...
ui/views/widget/widget_unittest.cc:1402: if (IsMus())
On 2016/06/01 17:18:27, sky wrote:
> Can you link this and the next to bugs with a TODO?

Done.

[sadrul, 2016-06-01 18:31:55]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-06-01 18:31:56]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2032573002/#ps40001
(title: ".")

[commit-bot: I haz the power, 2016-06-01 18:32:39]

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

[commit-bot: I haz the power, 2016-06-01 19:30:24]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-01 19:31:48]

Description was changed from

==========
views: Fix a couple of flaky tests on windows/linux, disable them on mus.

We can no longer expect a single cycle of message-loop iteration to
paint a widget, since painting is now driven by BeginFrameSource. So
update the tests to wait for the painting to happen first.

These tests create desktop-widgets, which we do not support in mus, so
disable the tests in mus.

BUG=596039
==========

to

==========
views: Fix a couple of flaky tests on windows/linux, disable them on mus.

We can no longer expect a single cycle of message-loop iteration to
paint a widget, since painting is now driven by BeginFrameSource. So
update the tests to wait for the painting to happen first.

These tests create desktop-widgets, which we do not support in mus, so
disable the tests in mus.

BUG=596039

Committed: https://crrev.com/3845af20c5138df223fd0fba3eb5a275aacea6b8
Cr-Commit-Position: refs/heads/master@{#397206}
==========

[commit-bot: I haz the power, 2016-06-01 19:31:49]

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