Clean up client state callback.

remoting/webapp/me2mom/viewer_plugin_proto.js

 // Copyright (c) 2011 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.
 
 // This file contains type definitions for the viewer plugin. It is used only
 // with JSCompiler to verify the type-correctness of our code.
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
@@ skipping 14 matching lines @@
 /**
  * @param {string} hostJid The host's JID.
  * @param {string} hostPublicKey The host's public key.
  * @param {string} clientJid The client's JID.
  * @param {string} accessCode The access code.
  * @return {void} Nothing.
 */
 remoting.ViewerPlugin.prototype.connect =
     function(hostJid, hostPublicKey, clientJid, accessCode) {};
 
+/** @type {function(number, number): void} State change callback function. */
+remoting.ViewerPlugin.prototype.connectionInfoUpdate;

[Wez, 2011/11/16 21:58:26]

Did we not need this before because of some default for function():void?

[Jamie, 2011/11/17 22:06:18]

It should have been there before, but the compiler didn't complain because it's
fine to write "object.undeclared_field = blah"

+
 /** @type {number} */ remoting.ViewerPlugin.prototype.apiMinVersion;
 /** @type {number} */ remoting.ViewerPlugin.prototype.apiVersion;
 
 /** @type {number} */ remoting.ViewerPlugin.prototype.desktopHeight;
 /** @type {number} */ remoting.ViewerPlugin.prototype.desktopWidth;
 
 /** @type {number} */ remoting.ViewerPlugin.prototype.status;
 /** @type {number} */ remoting.ViewerPlugin.prototype.error;
 
 /** @type {number} */ remoting.ViewerPlugin.prototype.STATUS_UNKNOWN;
@@ skipping 10 matching lines @@
     remoting.ViewerPlugin.prototype.ERROR_INCOMPATIBLE_PROTOCOL;
 /** @type {number} */ remoting.ViewerPlugin.prototype.ERROR_NETWORK_FAILURE;
 
 /** @type {number} */ remoting.ViewerPlugin.prototype.videoBandwidth;
 /** @type {number} */ remoting.ViewerPlugin.prototype.videoCaptureLatency;
 /** @type {number} */ remoting.ViewerPlugin.prototype.videoDecodeLatency;
 /** @type {number} */ remoting.ViewerPlugin.prototype.videoEncodeLatency;
 /** @type {number} */ remoting.ViewerPlugin.prototype.videoFrameRate;
 /** @type {number} */ remoting.ViewerPlugin.prototype.videoRenderLatency;
 /** @type {number} */ remoting.ViewerPlugin.prototype.roundTripLatency;