Hangout remote desktop part I - It2Me mode

remoting/webapp/client_screen.js

Index: remoting/webapp/client_screen.js
diff --git a/remoting/webapp/client_screen.js b/remoting/webapp/client_screen.js
index cb54aa358610d94f66dc65ff869488d962ad183a..73a887aa4d9e66ebfbc04a962009066fffd132d9 100644
--- a/remoting/webapp/client_screen.js
+++ b/remoting/webapp/client_screen.js
@@ -55,7 +55,7 @@ remoting.onVisibilityChanged = function() {
     remoting.clientSession.pauseVideo(
       ('hidden' in document) ? document.hidden : document.webkitHidden);
   }
-}
+};
 
 /**
  * Disconnect the remoting client.
@@ -89,6 +89,9 @@ function onClientStateChange_(state) {
       if (remoting.clientSession.getMode() ==
           remoting.ClientSession.Mode.IT2ME) {
         remoting.setMode(remoting.AppMode.CLIENT_SESSION_FINISHED_IT2ME);
+        remoting.HangoutSessionEvents.raiseEvent(
+            'sessionStateChanged',

[Jamie, 2014/08/07 23:12:35]

You've missed this and a few other places where this string is still hard-coded.

If you take my advice about re-using ClientSession.State, this becomes a lot
simpler, because you can raise the event outside the switch statement. You'll
fire more events that way, but since the recipient has to check the state
anyway, I don't think that matters.

[kelvinp, 2014/08/08 01:14:21]

Done.

+            remoting.HangoutSessionEvents.SessionStates.CLOSED);
       } else {
         remoting.setMode(remoting.AppMode.CLIENT_SESSION_FINISHED_ME2ME);
       }
@@ -133,6 +136,10 @@ function showConnectError_(errorTag) {
                                     : remoting.connector.getConnectionMode();
   if (mode == remoting.ClientSession.Mode.IT2ME) {
     remoting.setMode(remoting.AppMode.CLIENT_CONNECT_FAILED_IT2ME);
+    remoting.HangoutSessionEvents.raiseEvent(
+      'sessionStateChanged',
+      remoting.HangoutSessionEvents.SessionStates.ERROR
+    );
   } else {
     remoting.setMode(remoting.AppMode.CLIENT_CONNECT_FAILED_ME2ME);
   }
@@ -317,6 +324,10 @@ remoting.onConnected = function(clientSession) {
   remoting.toolbar.preview();
   remoting.clipboard.startSession();
   updateStatistics_();
+  remoting.HangoutSessionEvents.raiseEvent(
+    'sessionStateChanged',
+    remoting.HangoutSessionEvents.SessionStates.CONNECTED
+  );
   if (remoting.connector.pairingRequested) {
     /**
      * @param {string} clientId