Windows chromoting host installation via the NPAPI plugin

remoting/webapp/host_setup_dialog.js

 // Copyright (c) 2012 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.
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /**
@@ skipping 354 matching lines @@
     that.updateState_();
   }
 
   /** @param {remoting.HostController.State} state */
   var onHostState = function(state) {
     // Verify if the host has been installed. If not then try to prompt the user
     // again.
     var installed =
         state != remoting.HostController.State.NOT_INSTALLED &&
         state != remoting.HostController.State.INSTALLING;
-    if (installed) {
+
+    // On Windows we perform the host installation after showing the pin form.
+    if (installed || navigator.platform == 'Win32') {

[Sergey Ulanov, 2014/04/10 22:40:29]

I think on windows we still want to use hostInstallDialog() it's just it would
work differently there.

But currently this code should not be executed on Windows at all - see line 225.
Installation step is already skipped on Windows, so you don't need this change,
I think.

       that.flow_.switchToNextStep();
       that.updateState_();
     } else {
       hostInstallDialog.tryAgain();
     }
   }
 
   /** @type {remoting.HostInstallDialog} */
   var hostInstallDialog = new remoting.HostInstallDialog();
   hostInstallDialog.show(onDone, onError);
@@ skipping 172 matching lines @@
     var c = pin.charAt(i);
     if ((c < '0') || (c > '9')) {
       return false;
     }
   }
   return true;
 };
 
 /** @type {remoting.HostSetupDialog} */
 remoting.hostSetupDialog = null;