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

chrome/test/media_router/media_router_integration_ui_browsertest.cc

Index: chrome/test/media_router/media_router_integration_ui_browsertest.cc
diff --git a/chrome/test/media_router/media_router_integration_ui_browsertest.cc b/chrome/test/media_router/media_router_integration_ui_browsertest.cc
index 9193327541d9dbc89421dc7868b4258f8a26430e..e60235b6b502a66f4905c46babf6f1a5b2421947 100644
--- a/chrome/test/media_router/media_router_integration_ui_browsertest.cc
+++ b/chrome/test/media_router/media_router_integration_ui_browsertest.cc
@@ -81,6 +81,7 @@ IN_PROC_BROWSER_TEST_F(MediaRouterIntegrationBrowserTest, MANUAL_Dialog_Basic) {
       dialog_contents, sink_name_script);
   ASSERT_EQ(kTestSinkName, sink_name);
 
+#if defined(OS_MACOSX) || defined(OS_WIN)
   // Simulate moving the mouse off the dialog. Confirm that the dialog closes
   // automatically after the route is closed.
   // In tests, it sometimes takes too long to CloseRouteOnUI() to finish so
@@ -91,8 +92,11 @@ IN_PROC_BROWSER_TEST_F(MediaRouterIntegrationBrowserTest, MANUAL_Dialog_Basic) {
       "window.document.getElementById('media-router-container').dispatchEvent("
       "new Event('mouseleave')))");
   ASSERT_TRUE(content::ExecuteScript(dialog_contents, mouse_leave_script));
+#endif
   CloseRouteOnUI();
+#if defined(OS_MACOSX) || defined(OS_WIN)
   WaitUntilDialogClosed(web_contents);

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

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]

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.

+#endif
 }
 
 IN_PROC_BROWSER_TEST_F(MediaRouterIntegrationBrowserTest,