Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1388)

Unified Diff: content/browser/renderer_host/render_widget_host_impl.cc

Issue 10917102: Context menus should appear above the touch point if invoked by long press. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: patch Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_widget_host_impl.cc
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
index 0e6d1eefda5b5c6d2d49d422454201e93b9b5b72..2886c7136a526ba65e658a0e6c4712985eb007fa 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1007,6 +1007,8 @@ void RenderWidgetHostImpl::ForwardInputEvent(const WebInputEvent& input_event,
bool is_keyboard_shortcut) {
TRACE_EVENT0("renderer_host", "RenderWidgetHostImpl::ForwardInputEvent");
+ in_process_event_types_.push(input_event.type);
+
if (!process_->HasConnection())
return;
@@ -1530,6 +1532,9 @@ void RenderWidgetHostImpl::DidUpdateBackingStore(
void RenderWidgetHostImpl::OnMsgInputEventAck(WebInputEvent::Type event_type,
bool processed) {
TRACE_EVENT0("renderer_host", "RenderWidgetHostImpl::OnMsgInputEventAck");
+ DCHECK(!in_process_event_types_.empty() &&
+ in_process_event_types_.front() == event_type);
+ in_process_event_types_.pop();
// Log the time delta for processing an input event.
TimeDelta delta = TimeTicks::Now() - input_event_start_time_;

Powered by Google App Engine
This is Rietveld 408576698