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

remoting/host/resizing_host_observer.h

 // 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.
 
 #ifndef REMOTING_HOST_RESIZING_HOST_OBSERVER_H_
 #define REMOTING_HOST_RESIZING_HOST_OBSERVER_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "base/time.h"
 #include "remoting/host/screen_controls.h"
+#include "remoting/host/screen_resolution.h"
 #include "third_party/skia/include/core/SkSize.h"
 
 namespace remoting {
 
 class DesktopResizer;
-class ScreenResolution;
 
 // TODO(alexeypa): Rename this class to reflect that it is not
 // HostStatusObserver any more.
 
 // Uses the specified DesktopResizer to match host desktop size to the client
 // view size as closely as is possible. When the connection closes, restores
 // the original desktop size.
 class ResizingHostObserver : public ScreenControls {
  public:
   explicit ResizingHostObserver(scoped_ptr<DesktopResizer> desktop_resizer);
   virtual ~ResizingHostObserver();
 
   // ScreenControls interface.
   virtual void SetScreenResolution(const ScreenResolution& resolution) OVERRIDE;
 
  private:
+  void SetPendingScreenResolution();
+
   scoped_ptr<DesktopResizer> desktop_resizer_;
   SkISize original_size_;
 
+  ScreenResolution pending_resolution_;

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

nit: Add comments explaining what these members are for.

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

Done.

+  base::Time previous_resize_time_;
+  bool resize_pending_;
+
+  base::WeakPtrFactory<ResizingHostObserver> weak_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(ResizingHostObserver);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_RESIZING_HOST_OBSERVER_H_