Fixing the Remote Desktop Browser Test (Me2Me connection test) Failure

chrome/test/remoting/remote_desktop_browsertest.cc

Index: chrome/test/remoting/remote_desktop_browsertest.cc
diff --git a/chrome/test/remoting/remote_desktop_browsertest.cc b/chrome/test/remoting/remote_desktop_browsertest.cc
index e1f52288c07f880a48dcd57079f06ef77d566bf3..258c5aaabd6c475f5db0a147269b69e85b19dd48 100644
--- a/chrome/test/remoting/remote_desktop_browsertest.cc
+++ b/chrome/test/remoting/remote_desktop_browsertest.cc
@@ -588,7 +588,7 @@ void RemoteDesktopBrowserTest::ConnectToRemoteHost(
   std::string element_id = "host_" + host_id;
 
   ConditionalTimeoutWaiter hostOnlineWaiter(

[joedow, 2016/01/20 22:23:06]

You may want to add a comment about what this code is doing (3 attempts at 20
seconds each) since it caused some confusion earlier.

[alog1, 2016/01/20 22:39:54]

Done.

-      base::TimeDelta::FromSeconds(30), base::TimeDelta::FromSeconds(5),
+      base::TimeDelta::FromSeconds(60), base::TimeDelta::FromSeconds(20),
       base::Bind(&RemoteDesktopBrowserTest::IsHostOnline,
                  base::Unretained(this), host_id));
   EXPECT_TRUE(hostOnlineWaiter.Wait());
@@ -820,14 +820,10 @@ void RemoteDesktopBrowserTest::WaitForConnection() {
 }
 
 bool RemoteDesktopBrowserTest::IsHostOnline(const std::string& host_id) {
-  bool refresh_host_list =
-      ExecuteScriptAndExtractBool("remoting.hostList.refreshAndDisplay()");
 
-  if (!refresh_host_list) {
-    return false;
-  }
+  ExecuteScript("remoting.hostList.refreshAndDisplay()");
 
-  // Verify the host is online.
+ // Verify the host is online.
   std::string element_id = "host_" + host_id;
   std::string host_div_class = ExecuteScriptAndExtractString(
       "document.getElementById('" + element_id + "').parentNode.className");