Add host-side rate-limiting to desktop resize events.

remoting/webapp/client_session.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 handling creation and teardown of a remoting client session.
  *
  * The ClientSession class controls lifetime of the client plugin
  * object and provides the plugin with the functionality it needs to
@@ skipping 867 matching lines @@
   }
 
   // Defer notifying the host of the change until the window stops resizing, to
   // avoid overloading the control channel with notifications.
   if (this.resizeToClient_) {
     this.notifyClientResolutionTimer_ = window.setTimeout(
         this.plugin.notifyClientResolution.bind(this.plugin,
                                                 window.innerWidth,
                                                 window.innerHeight,
                                                 window.devicePixelRatio),
-        1000);
+        250);

[Jamie, 2013/06/03 23:26:28]

250ms seems to be a good compromise between sending the resize command as soon
as possible after the user stops dragging the window border and not sending
erroneous resize commands while s/he is still doing so (sending resize commands
too early is bad because it causes the host-side rate-limiting to kick in,
delaying resizing the desktop to the intended size).

[alexeypa (please no reviews), 2013/06/04 00:05:26]

Consider using different timeout depending on the kind of the host we are
connected. The timeout should be shorter for the host implementing
rate-limiting. 

A new capability needs to be added to the host and webapp to be able to do that.
Look for 'sendInitialResolution' to see how it is done. The only caveat is that
this new capability should be returned by all implementation of
DesktopEnvironment::GetCapabilities().

[Jamie, 2013/06/04 01:27:49]

Done.

   }
 
   // If bump-scrolling is enabled, adjust the plugin margins to fully utilize
   // the new window area.
   this.scroll_(0, 0);
 };
 
 /**
  * Requests that the host pause or resume video updates.
  *
@@ skipping 305 matching lines @@
       var lateAdjustment = 1 + (now - expected) / timeout;
       if (!that.scroll_(lateAdjustment * dx, lateAdjustment * dy)) {
         that.bumpScrollTimer_ = window.setTimeout(
             function() { repeatScroll(now + timeout); },
             timeout);
       }
     };
     repeatScroll(new Date().getTime());
   }
 };