Use showConnectError_ instead of remoting.showError to correctly distinguish between connection modes and reset web-app state.

BUG=175008


Review URL: https://chromiumcodereview.appspot.com/12220080

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181591 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 14 insertions, 3 deletions

diff --git a/remoting/webapp/client_screen.js b/remoting/webapp/client_screen.js
index 1140dc7..4e1952d 100644
--- a/remoting/webapp/client_screen.js
+++ b/remoting/webapp/client_screen.js
@@ -31,7 +31,7 @@ remoting.connectIT2Me = function() {
   remoting.connector = new remoting.SessionConnector(
       document.getElementById('session-mode'),
       remoting.onConnected,
-      remoting.showErrorMessage);
+      showConnectError_);
   var accessCode = document.getElementById('access-code-entry').value;
   remoting.setMode(remoting.AppMode.CLIENT_CONNECTING);
   remoting.connector.connectIT2Me(accessCode);
@@ -152,7 +152,9 @@ function showConnectError_(errorTag) {
   var errorDiv = document.getElementById('connect-error-message');
   l10n.localizeElementFromTag(errorDiv, /** @type {string} */ (errorTag));
   remoting.accessCode = '';
-  if (remoting.clientSession.mode == remoting.ClientSession.Mode.IT2ME) {
+  var mode = remoting.clientSession ? remoting.clientSession.mode
+                                    : remoting.connector.getConnectionMode();
+  if (mode == remoting.ClientSession.Mode.IT2ME) {
     remoting.setMode(remoting.AppMode.CLIENT_CONNECT_FAILED_IT2ME);
   } else {
     remoting.setMode(remoting.AppMode.CLIENT_CONNECT_FAILED_ME2ME);
@@ -209,7 +211,7 @@ remoting.connectMe2Me = function(hostId) {
   remoting.connector = new remoting.SessionConnector(
       document.getElementById('session-mode'),
       remoting.onConnected,
-      remoting.showErrorMessage);
+      showConnectError_);
   /** @type {Element} */
   var pinForm = document.getElementById('pin-form');
   /** @param {Event} event */
diff --git a/remoting/webapp/session_connector.js b/remoting/webapp/session_connector.js
index 0cb9105..7816144 100644
--- a/remoting/webapp/session_connector.js
+++ b/remoting/webapp/session_connector.js
@@ -166,6 +166,15 @@ remoting.SessionConnector.prototype.cancel = function() {
 };
 
 /**
+ * Get the connection mode (Me2Me or IT2Me)
+ *
+ * @return {remoting.ClientSession.Mode}
+ */
+remoting.SessionConnector.prototype.getConnectionMode = function() {
+  return this.connectionMode_;
+};
+
+/**
  * Continue an IT2Me connection once an access token has been obtained.
  *
  * @param {string} token An OAuth2 access token.