[Webapp Refactor] Remove It2Me connection code from SessionConnectorImpl.

remoting/webapp/crd/js/crd_connect.js

Index: remoting/webapp/crd/js/crd_connect.js
diff --git a/remoting/webapp/crd/js/crd_connect.js b/remoting/webapp/crd/js/crd_connect.js
index ea4f7a83f1315de22ee7429707daadcac9149fc7..c21fd2cc6786981fb1436375d0a40b4d55f8b031 100644
--- a/remoting/webapp/crd/js/crd_connect.js
+++ b/remoting/webapp/crd/js/crd_connect.js
@@ -19,7 +19,12 @@ remoting.connectIT2Me = function() {
   var connector = remoting.app.getSessionConnector();
   var accessCode = document.getElementById('access-code-entry').value;
   remoting.setMode(remoting.AppMode.CLIENT_CONNECTING);
-  connector.connectIT2Me(accessCode);
+  remoting.It2MeConnectFlow.start(connector, accessCode).catch(
+    function(reason){

[garykac, 2015/03/06 22:27:27]

formatting nit: What do you think about starting this line with 'catch' to make
it more obvious that this is error-handling code?

remoting.It2MeConnectFlow.start(connector, accessCode).
    catch(function(reason){
      var errorDiv = document.getElementById('connect-error-message');
      l10n.localizeElementFromTag(errorDiv, /** @type {string} */ (reason));
      remoting.setMode(remoting.AppMode.CLIENT_CONNECT_FAILED_IT2ME);
    });

(frankly, I would prefer the line to start with '.catch', but Chrome style wants
'.'s at the end of the line :-( )

[kelvinp, 2015/03/06 23:33:51]

Done.

+      var errorDiv = document.getElementById('connect-error-message');
+      l10n.localizeElementFromTag(errorDiv, /** @type {string} */ (reason));
+      remoting.setMode(remoting.AppMode.CLIENT_CONNECT_FAILED_IT2ME);
+  });
 };
 
 /**