Throttle SelectRange IPC

content/renderer/render_view_impl.cc

 // 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.
 
 #include "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 1254 matching lines @@
 }
 
 void RenderViewImpl::OnSelectRange(const gfx::Point& start,
                                    const gfx::Point& end) {
   if (!webview())
     return;
 
   handling_select_range_ = true;
   webview()->focusedFrame()->selectRange(start, end);
   handling_select_range_ = false;
+  Send(new ViewHostMsg_SelectRange_ACK(routing_id_));

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

nit: I suggest sending the ACK before calling selectRange(). That way you allow
the next SelectRange IPC to overlap slightly with the processing of the current
SelectRange IPC.  In other cases, such an approach has been useful.

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

Done.

 }
 
 void RenderViewImpl::OnSetHistoryLengthAndPrune(int history_length,
                                                 int32 minimum_page_id) {
   DCHECK(history_length >= 0);
   DCHECK(history_list_offset_ == history_list_length_ - 1);
   DCHECK(minimum_page_id >= -1);
 
   // Generate the new list.
   std::vector<int32> new_history_page_ids(history_length, -1);
@@ skipping 4435 matching lines @@
 bool RenderViewImpl::WebWidgetHandlesCompositorScheduling() const {
   return !!RenderThreadImpl::current()->compositor_thread();
 }
 
 void RenderViewImpl::OnJavaBridgeInit() {
   DCHECK(!java_bridge_dispatcher_);
 #if defined(ENABLE_JAVA_BRIDGE)
   java_bridge_dispatcher_ = new JavaBridgeDispatcher(this);
 #endif
 }