Handle host need upgrade in Pin browser test

remoting/webapp/browser_test/update_pin_browser_test.js

Index: remoting/webapp/browser_test/update_pin_browser_test.js
diff --git a/remoting/webapp/browser_test/update_pin_browser_test.js b/remoting/webapp/browser_test/update_pin_browser_test.js
index 0d0eba5fa40a164de3480ce57240d798514e1f7e..50df331d75818670d2f02a6ae6464c71b225514b 100644
--- a/remoting/webapp/browser_test/update_pin_browser_test.js
+++ b/remoting/webapp/browser_test/update_pin_browser_test.js
@@ -27,19 +27,19 @@ browserTest.Update_PIN.prototype.run = function(data) {
                      'The new PIN and the old PIN cannot be the same');
 
   this.changePIN_(data.new_pin).then(
-    this.connect_.bind(this)
+    browserTest.connectMe2Me
   ).then(
     this.enterPIN_.bind(this, data.old_pin, true /* expectError*/)
   ).then(
     // Sleep for two seconds to allow for the login backoff logic to reset.
     base.Promise.sleep.bind(null, LOGIN_BACKOFF_WAIT)
   ).then(
-    this.connect_.bind(this)
+    browserTest.connectMe2Me
   ).then(
     this.enterPIN_.bind(this, data.new_pin, false /* expectError*/)
   ).then(
     // Clean up the test by disconnecting and changing the PIN back
-    this.disconnect_.bind(this)
+    browserTest.connectMe2Me

[Jamie, 2014/07/11 19:33:22]

Is this a copy/paste error? The previous code was disconnecting, but the new
code is connecting.

[kelvinp, 2014/07/11 21:11:22]

My bad.  Good catch.

   ).then(
     // The PIN must be restored regardless of success or failure.
     this.changePIN_.bind(this, data.old_pin),
@@ -72,11 +72,6 @@ browserTest.Update_PIN.prototype.changePIN_ = function(newPin) {
   });
 };
 
-browserTest.Update_PIN.prototype.connect_ = function() {
-  browserTest.clickOnControl('this-host-connect');
-  return browserTest.onUIMode(remoting.AppMode.CLIENT_PIN_PROMPT);
-};
-
 browserTest.Update_PIN.prototype.disconnect_ = function() {
   var AppMode = remoting.AppMode;