Download the host components when user tries to enable Me2Me host.

remoting/webapp/host_native_messaging.js

 // Copyright 2013 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
  * Class to communicate with the Host components via Native Messaging.
  */
 
 'use strict';
@@ skipping 47 matching lines @@
  * Sets up connection to the Native Messaging host process and exchanges
  * 'hello' messages. If Native Messaging is not available or the host
  * process is not installed, this returns false to the callback.
  *
  * @param {function(): void} onDone Called after successful initialization.
  * @param {function(remoting.Error): void} onError Called if initialization
  *     failed.
  * @return {void} Nothing.
  */
 remoting.HostNativeMessaging.prototype.initialize = function(onDone, onError) {
-  if (!chrome.runtime.connectNative) {

[weitao, 2014/01/31 18:37:11]

I am about to commit a CL that remove this code and add a minimum required
version to our webapp. You will need to sync and merge the conflict.

[Sergey Ulanov, 2014/01/31 23:34:17]

Done.

-    console.log('Native Messaging API not available');
-    onError(remoting.Error.UNEXPECTED);
-    return;
-  }
-
-  // NativeMessaging API exists on Chrome 26.xxx but fails to notify
-  // onDisconnect in the case where the Host components are not installed. Need
-  // to blacklist these versions of Chrome.
-  var majorVersion = navigator.appVersion.match('Chrome/(\\d+)\.')[1];
-  if (!majorVersion || majorVersion <= 26) {
-    console.log('Native Messaging not supported on this version of Chrome');
-    onError(remoting.Error.UNEXPECTED);
-    return;
-  }
-
+  console.log("NM Init");

[Jamie, 2014/01/31 18:39:15]

Are these logs needed? If so, they should be a bit more descriptive than this.

In fact, it's not clear what the changes to this file have to do with this CL.
Can you be done separately?

[Sergey Ulanov, 2014/01/31 23:34:17]

Reverted the changes in this method, but the fix in onDisconnect_() needs to be
in this CL because now pendingReplies_ may be modified while onDisconnect_()
call one of the pendingReplies_.

   try {
+  console.log("NM Init2 " + this.pendingReplies_);
+    this.pendingReplies_ = {};
     this.port_ = chrome.runtime.connectNative(
         'com.google.chrome.remote_desktop');
     this.port_.onMessage.addListener(this.onIncomingMessage_.bind(this));
     this.port_.onDisconnect.addListener(this.onDisconnect_.bind(this));
     this.postMessage_({type: 'hello'}, onDone,
                       onError.bind(null, remoting.Error.UNEXPECTED));
+  console.log("NM Init3");
   } catch (err) {
     console.log('Native Messaging initialization failed: ',
                 /** @type {*} */ (err));
     onError(remoting.Error.UNEXPECTED);
     return;
   }
 };
 
 /**
  * Verifies that |object| is of type |type|, logging an error if not.
@@ skipping 273 matching lines @@
 };
 
 /**
  * @return {void} Nothing.
  * @private
  */
 remoting.HostNativeMessaging.prototype.onDisconnect_ = function() {
   console.error('Native Message port disconnected');
 
   // Notify the error-handlers of any requests that are still outstanding.
-  for (var id in this.pendingReplies_) {
-    this.pendingReplies_[/** @type {number} */(id)].onError(
+  var pendingReplies = this.pendingReplies_;
+  this.pendingReplies_ = {};
+
+  for (var id in pendingReplies) {
+    pendingReplies[/** @type {number} */(id)].onError(
         remoting.Error.UNEXPECTED);
   }
-  this.pendingReplies_ = {};
 }
 
 /**
  * @param {function(string):void} onDone Callback to be called with the
  *     local hostname.
  * @param {function(remoting.Error):void} onError The callback to be triggered
  *     on error.
  * @return {void} Nothing.
  */
 remoting.HostNativeMessaging.prototype.getHostName =
@@ skipping 206 matching lines @@
  *     on error.
  * @return {void} Nothing.
  */
 remoting.HostNativeMessaging.prototype.getCredentialsFromAuthCode =
     function(authorizationCode, onDone, onError) {
   this.postMessage_({
     type: 'getCredentialsFromAuthCode',
     authorizationCode: authorizationCode
   }, onDone, onError);
 };