[Media Router] Remove second dialog state check in Dialog_Basic integration test on Linux.

[Media Router] Remove second dialog state check in Dialog_Basic integration test on Linux.

BUG=577943
Committed: https://crrev.com/67979c2e029e2599433a2ec37d85eee114f44d97
Cr-Commit-Position: refs/heads/master@{#370793}

[apacible, 2016-01-21 19:05:40]

Description was changed from

==========
fix integration test

BUG=
==========

to

==========
[Media Router] Remove second dialog state check in Dialog_Basic integration test
on Linux.

BUG=577943
==========

[apacible, 2016-01-21 19:05:40]

apacible@chromium.org changed reviewers:
+ leilei@chromium.org

[apacible, 2016-01-21 19:06:17]

PTAL, thanks!

[Lei Lei, 2016-01-21 19:07:55]

lgtm

https://codereview.chromium.org/1619663004/diff/1/chrome/test/media_router/me...
File chrome/test/media_router/media_router_integration_ui_browsertest.cc
(right):

https://codereview.chromium.org/1619663004/diff/1/chrome/test/media_router/me...
chrome/test/media_router/media_router_integration_ui_browsertest.cc:98:
WaitUntilDialogClosed(web_contents);
I think we should keep this for Linux, because the dialog will be closed
automatically if there is no user action after 3s, is it?

[apacible, 2016-01-21 19:10:02]

https://codereview.chromium.org/1619663004/diff/1/chrome/test/media_router/me...
File chrome/test/media_router/media_router_integration_ui_browsertest.cc
(right):

https://codereview.chromium.org/1619663004/diff/1/chrome/test/media_router/me...
chrome/test/media_router/media_router_integration_ui_browsertest.cc:98:
WaitUntilDialogClosed(web_contents);
On 2016/01/21 19:07:55, Lei Lei wrote:
> I think we should keep this for Linux, because the dialog will be closed
> automatically if there is no user action after 3s, is it?

I removed this for Linux since it appears mouseenter/mouseleave event
dispatching (not explicitly called by us) isn't consistent and so this could
turn the test flaky.

[apacible, 2016-01-21 19:12:32]

The CQ bit was checked by apacible@chromium.org

[commit-bot: I haz the power, 2016-01-21 19:14:21]

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

[commit-bot: I haz the power, 2016-01-21 20:15:35]

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

[commit-bot: I haz the power, 2016-01-21 20:15:35]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[apacible, 2016-01-21 20:29:06]

The CQ bit was checked by apacible@chromium.org

[commit-bot: I haz the power, 2016-01-21 20:30:47]

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

[commit-bot: I haz the power, 2016-01-21 21:39:31]

Description was changed from

==========
[Media Router] Remove second dialog state check in Dialog_Basic integration test
on Linux.

BUG=577943
==========

to

==========
[Media Router] Remove second dialog state check in Dialog_Basic integration test
on Linux.

BUG=577943
==========

[commit-bot: I haz the power, 2016-01-21 21:39:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-21 21:40:29]

Description was changed from

==========
[Media Router] Remove second dialog state check in Dialog_Basic integration test
on Linux.

BUG=577943
==========

to

==========
[Media Router] Remove second dialog state check in Dialog_Basic integration test
on Linux.

BUG=577943

Committed: https://crrev.com/67979c2e029e2599433a2ec37d85eee114f44d97
Cr-Commit-Position: refs/heads/master@{#370793}
==========

[commit-bot: I haz the power, 2016-01-21 21:40:30]

Patchset 1 (id:??) landed as
https://crrev.com/67979c2e029e2599433a2ec37d85eee114f44d97
Cr-Commit-Position: refs/heads/master@{#370793}