[Chromoting] Fix jscompiler warnings in client_plugin_async.js

remoting/webapp/client_plugin_async.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.
 
 /**
  * @fileoverview
  * Class that wraps low-level details of interacting with the client plugin.
  *
  * This abstracts a <embed> element and controls the plugin which does
  * the actual remoting work. It also handles differences between
@@ skipping 185 matching lines @@
     }
   } else if (message.method == 'onFirstFrameReceived') {
     if (remoting.clientSession) {
       remoting.clientSession.onFirstFrameReceived();
     }
   } else if (message.method == 'onConnectionReady') {
     if (typeof message.data['ready'] != 'boolean') {
       console.error('Received incorrect onConnectionReady message.');
       return;
     }
-    this.onConnectionReadyHandler(message.data['ready']);
+    var ready = /** @type {boolean} */ message.data['ready'];
+    this.onConnectionReadyHandler(ready);

[Sergey Ulanov, 2012/07/23 21:39:34]

Might be better to write like this:
  this.onConnectionReadyHandler(
      /** @type {boolean} */ message.data['ready']);

[garykac, 2012/07/23 22:45:25]

I tried that first and it didn't work. Not sure why the jscompiler didn't like
it.

   }
 }
 
 /**
  * Deletes the plugin.
  */
 remoting.ClientPluginAsync.prototype.cleanup = function() {
   this.plugin.parentNode.removeChild(this.plugin);
 };
 
@@ skipping 193 matching lines @@
     this.plugin.width = width;
     this.plugin.height = height;
     // Center the plugin just underneath the "Connnecting..." dialog.
     var parentNode = this.plugin.parentNode;
     var dialog = document.getElementById('client-dialog');
     var dialogRect = dialog.getBoundingClientRect();
     parentNode.style.top = (dialogRect.bottom + 16) + 'px';
     parentNode.style.left = (window.innerWidth - width) / 2 + 'px';
   }
 };