Hangouts remote desktop part III - It2MeService

remoting/webapp/background/background.js

 // Copyright 2014 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.
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 (function(){
 
 /** @return {boolean} */
@@ skipping 25 matching lines @@
     });
     chrome.contextMenus.onClicked.addListener(onContextMenu);
   }
 
   initializeContextMenu();
   chrome.app.runtime.onLaunched.addListener(
       appLauncher.launch.bind(appLauncher)
   );
 }
 
+/**
+ * The background service is responsible for listening to incoming connection
+ * requests from Hangouts and the webapp.
+ *
+ * @param {remoting.AppLauncher} appLauncher
+ */
+ function initializeBackgroundService(appLauncher) {
+   /** @type {remoting.It2MeService} */
+   var it2meService = new remoting.It2MeService(appLauncher);
+   it2meService.init();
+   remoting.it2meService = it2meService;
+
+   chrome.runtime.onSuspend.addListener(function() {
+     it2meService.dispose();

[Jamie, 2014/08/13 01:44:52]

Reset remoting.it2meService to null? In fact, it might be clearer if this was
something like:

if (remoting.it2meService) {
  remoting.it2meService.dispose();
  remoting.it2meService = null;
}

ie, get rid of the temporary altogether.

[kelvinp, 2014/08/13 23:44:09]

Done.

+   });
+ }
+
 function main() {
   /** @type {remoting.AppLauncher} */
   var appLauncher = new remoting.V1AppLauncher();
   if (isAppsV2()) {
     appLauncher = new remoting.V2AppLauncher();
     initializeAppV2(appLauncher);
   }
+  initializeBackgroundService(appLauncher);
 }
 
 window.addEventListener('load', main, false);
 
 }());