Enable PIN browser tests on the waterfall.

remoting/webapp/browser_test/browser_test.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
  *
  * Provides basic functionality for JavaScript based browser test.
  *
  * To define a browser test, create a class under the browserTest namespace.
@@ skipping 172 matching lines @@
     }
     remoting.testEvents.addEventListener(uiModeChanged, onUIModeChanged);
   });
 };
 
 /**
  * @return {Promise}
  */
 browserTest.connectMe2Me = function() {
   var AppMode = remoting.AppMode;
-  browserTest.clickOnControl('this-host-connect');
-  return browserTest.onUIMode(AppMode.CLIENT_HOST_NEEDS_UPGRADE).then(
-    function() {
+  // The 1 second is necessary because the click handler of 'this-host-connect'
+  // is hooked asynchronously.

[Jamie, 2015/02/23 20:56:26]

It's not entirely clear what this means. I suggest something like "...because
the event handler for 'this-host-connect' is registered asynchronously."

+  return base.Promise.sleep(1000).then(function() {
+      browserTest.clickOnControl('this-host-connect');
+    }).then(function(){
+      return browserTest.onUIMode(AppMode.CLIENT_HOST_NEEDS_UPGRADE);
+    }).then(function() {
       // On fulfilled.
       browserTest.clickOnControl('host-needs-update-connect-button');
     }, function() {
       // On time out.
       return Promise.resolve();
     }).then(function() {
       return browserTest.onUIMode(AppMode.CLIENT_PIN_PROMPT, 10000);
     });
 };
 
@@ skipping 100 matching lines @@
     var errorMsg = errorDiv.innerText;
     return Promise.reject('Unexpected error - ' + errorMsg);
   });
   return Promise.race([onConnected, onFailure]);
 };
 
 /**
  * @param {base.EventSource} eventSource
  * @param {string} event
  * @param {number} timeoutMs
- * @param {?string} opt_expectedData
+ * @param {*=} opt_expectedData
  * @return {Promise}
  */
 browserTest.expectEvent = function(eventSource, event, timeoutMs,
                                    opt_expectedData) {
   return new Promise(function(fullfil, reject) {
     /** @param {string=} actualData */
     var verifyEventParameters = function(actualData) {
       if (opt_expectedData === undefined || opt_expectedData === actualData) {
         fullfil(true);
       } else {
@@ skipping 49 matching lines @@
     console.log('browserTest: PIN Setup is done.');
     browserTest.clickOnControl('host-config-done-dismiss');
 
     // On Linux, we restart the host after changing the PIN, need to sleep
     // for ten seconds before the host is ready for connection.
     return base.Promise.sleep(HOST_SETUP_WAIT);
   });
 };
 
 /**
- * @return {Promise}
+ * @return {Promise<boolean>}
  */
 browserTest.isLocalHostStarted = function() {
   return new Promise(function(resolve) {
     remoting.hostController.getLocalHostState(function(state) {
       resolve(remoting.HostController.State.STARTED == state);
     });
   });
 };
 
 /**
@@ skipping 24 matching lines @@
  */
 browserTest.ensureRemoteConnectionEnabled = function(pin) {
   browserTest.ensureHostStartedWithPIN(pin).then(function() {
     browserTest.pass();
   }, function(reason) {
     browserTest.fail(reason);
   });
 };
 
 browserTest.init();