Don't transition to in-session mode until the connection has been established.

remoting/webapp/me2mom/client_session.js

 // Copyright (c) 2011 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
  * Session class that handles creation and teardown of a remoting session.
  *
  * This abstracts a <embed> element and controls the plugin which does the
  * actual remoting work.  There should be no UI code inside this class.  It
  * should be purely thought of as a controller of sorts.
  */
 
 'use strict';
 
 var remoting = remoting || {};
 
 (function() {
 /**
  * @param {string} hostJid The jid of the host to connect to.
  * @param {string} hostPublicKey The base64 encoded version of the host's
  *     public key.
  * @param {string} accessCode The access code for the IT2Me connection.
  * @param {string} email The username for the talk network.
  * @param {function(remoting.ClientSession.State):void} onStateChange
- *     The callback to invoke when the session changes state.
+ *     The callback to invoke when the session changes state. This callback
+ *     occurs after the state changes and is passed the previous state; the
+ *     new state is accessible via ClientSession's |state| property.
  * @constructor
  */
 remoting.ClientSession = function(hostJid, hostPublicKey, accessCode, email,
                                   onStateChange) {
   this.state = remoting.ClientSession.CREATED;
 
   this.hostJid = hostJid;
   this.hostPublicKey = hostPublicKey;
   this.accessCode = accessCode;
   this.email = email;
@@ skipping 67 matching lines @@
  * @param {Element} container The element to add the plugin to.
  * @param {string} oauth2AccessToken A valid OAuth2 access token.
  * @return {void} Nothing.
  */
 remoting.ClientSession.prototype.createPluginAndConnect =
     function(container, oauth2AccessToken) {
   this.plugin = document.createElement('embed');
   this.plugin.id = this.PLUGIN_ID;
   this.plugin.src = 'about://none';
   this.plugin.type = 'pepper-application/x-chromoting';
+  this.plugin.width = 0;
+  this.plugin.height = 0;
   container.appendChild(this.plugin);
 
   if (!this.isPluginVersionSupported_(this.plugin)) {
     // TODO(ajwong): Remove from parent.
     delete this.plugin;
     this.setState_(remoting.ClientSession.BAD_PLUGIN_VERSION);
     return;
   }
 
   var that = this;
@@ skipping 189 matching lines @@
   } else if (state == this.plugin.STATUS_FAILED) {
     this.setState_(remoting.ClientSession.State.CONNECTION_FAILED);
   }
 };
 
 /**
  * @param {remoting.ClientSession.State} state The new state for the session.
  * @return {void} Nothing.
  */
 remoting.ClientSession.prototype.setState_ = function(state) {
+  var oldState = state;
   this.state = state;
   if (this.onStateChange) {
-    this.onStateChange(this.state);
+    this.onStateChange(oldState);
   }
 };
 
 /**
  * This is a callback that gets called when the desktop size contained in the
  * the plugin has changed.
  *
  * @return {void} Nothing.
  */
 remoting.ClientSession.prototype.onDesktopSizeChanged_ = function() {
@@ skipping 61 matching lines @@
     'video_bandwidth': this.plugin.videoBandwidth,
     'capture_latency': this.plugin.videoCaptureLatency,
     'encode_latency': this.plugin.videoEncodeLatency,
     'decode_latency': this.plugin.videoDecodeLatency,
     'render_latency': this.plugin.videoRenderLatency,
     'roundtrip_latency': this.plugin.roundTripLatency
   };
 };
 
 }());