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 b6544af532fe3fb8696ddbd9193bf3fbcacc8683..c0e89d09f7c5a13f4a6f12409129f6d04c11bd9e 100644
--- a/content/browser/renderer_host/render_widget_host_impl.h
+++ b/content/browser/renderer_host/render_widget_host_impl.h
@@ -552,6 +552,8 @@ class CONTENT_EXPORT RenderWidgetHostImpl : virtual public RenderWidgetHost,
   void OnMsgDestroyPluginContainer(gfx::PluginWindowHandle id);
 #endif
 
+  void OnMsgSelectRangeAck();

[darin (slow to review), 2012/08/02 06:30:20]

nit: it seems like this should be listed after OnMsgBeginSmoothScroll since you
place it after that function in the .cc file.

[Iain Merrick, 2012/08/02 10:25:42]

Done.

+
   // Called (either immediately or asynchronously) after we're done with our
   // BackingStore and can send an ACK to the renderer so it can paint onto it
   // again.
@@ -687,6 +689,12 @@ class CONTENT_EXPORT RenderWidgetHostImpl : virtual public RenderWidgetHost,
   // are coalesced by merging deltas in a similar fashion as wheel events.
   GestureEventQueue coalesced_gesture_events_;
 
+  // Throttling text selection IPC, allowing at most one unACKed IPC in flight.
+  gfx::Point pend_select_start_;

[darin (slow to review), 2012/08/02 06:30:20]

nit: please use words.  pend -> pending?

[Iain Merrick, 2012/08/02 10:25:42]

Done. In fact I'll rename things a bit because using 'pending' in two different
senses seems very confusing.

+  gfx::Point pend_select_end_;
+  bool select_range_in_flight_;
+  bool select_range_pending_;
+
   // The time when an input event was sent to the RenderWidget.
   base::TimeTicks input_event_start_time_;