Throttle SelectRange IPC

content/browser/renderer_host/render_widget_host_impl.h

Index: content/browser/renderer_host/render_widget_host_impl.h
diff --git a/content/browser/renderer_host/render_widget_host_impl.h b/content/browser/renderer_host/render_widget_host_impl.h
index b7a528e2104f2a476d8331b6cda2eaa63825b938..1275fa3c52f418b7974f0b2485c7b0aacf12747b 100644
--- a/content/browser/renderer_host/render_widget_host_impl.h
+++ b/content/browser/renderer_host/render_widget_host_impl.h
@@ -7,6 +7,7 @@
 
 #include <deque>
 #include <string>
+#include <utility>
 #include <vector>
 
 #include "base/callback.h"
@@ -505,6 +506,7 @@ class CONTENT_EXPORT RenderWidgetHostImpl : virtual public RenderWidgetHost,
   void OnMsgInputEventAck(WebKit::WebInputEvent::Type event_type,
                           bool processed);
   void OnMsgBeginSmoothScroll(bool scroll_down, bool scroll_far);
+  void OnMsgSelectRangeAck();
   virtual void OnMsgFocus();
   virtual void OnMsgBlur();
   void OnMsgHasTouchEventHandlers(bool has_handlers);
@@ -686,6 +688,12 @@ class CONTENT_EXPORT RenderWidgetHostImpl : virtual public RenderWidgetHost,
   // are coalesced by merging deltas in a similar fashion as wheel events.
   GestureEventQueue coalesced_gesture_events_;
 
+  // (Similar to |mouse_move_pending_|.) True while waiting for SelectRange_ACK.
+  bool select_range_pending_;
+
+  // (Similar to |next_mouse_move_|.) The next SelectRange to send, if any.
+  scoped_ptr<std::pair<gfx::Point, gfx::Point> > next_select_range_;

[darin (slow to review), 2012/08/02 16:37:17]

Please just define a struct instead of using std::pair.  The usage of 'first'
and 'second' in code is rarely readable.

How about this:
  struct SelectionRange {
    gfx::Point start, end;
  };
  scoped_ptr<SelectionRange> next_selection_range_;

^^^ reads better to say that "SelectRange sends a selection range," which is why
I named the type SelectionRange instead of SelectRange.

[Iain Merrick, 2012/08/06 11:28:45]

Done.

+
   // The time when an input event was sent to the RenderWidget.
   base::TimeTicks input_event_start_time_;