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

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 727262923d6e425bcdad12f3d1ee3836b147f36d..0b031c0299ac5f1e238e76ce925a061cfead9519 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1011,6 +1011,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;
@@ -1534,6 +1536,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