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

Unified Diff: content/browser/frame_host/cross_process_frame_connector.cc

Issue 1993383002: Adding Event.Latency.Browser.WheelUI, Passing latency_info to ProcessMouse/MouseWheelEvent functions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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/frame_host/cross_process_frame_connector.cc
diff --git a/content/browser/frame_host/cross_process_frame_connector.cc b/content/browser/frame_host/cross_process_frame_connector.cc
index f16ce70104142bdb79b0851987aef05f084ca063..0a8b4e099186cb9777f4d2da6a748340d910ff17 100644
--- a/content/browser/frame_host/cross_process_frame_connector.cc
+++ b/content/browser/frame_host/cross_process_frame_connector.cc
@@ -172,7 +172,10 @@ void CrossProcessFrameConnector::BubbleScrollEvent(
memcpy(&resent_wheel_event, &event, sizeof(resent_wheel_event));
resent_wheel_event.x += offset_from_parent.x();
resent_wheel_event.y += offset_from_parent.y();
- parent_view->ProcessMouseWheelEvent(resent_wheel_event);
+ // TODO(wjmaclean): Initialize latency info correctly for OOPIFs.
+ // https://crbug.com/613628
+ ui::LatencyInfo latency_info;
+ parent_view->ProcessMouseWheelEvent(resent_wheel_event, latency_info);
} else {
NOTIMPLEMENTED();
}
@@ -217,13 +220,20 @@ void CrossProcessFrameConnector::OnForwardInputEvent(
}
if (blink::WebInputEvent::isMouseEventType(event->type)) {
- view_->ProcessMouseEvent(*static_cast<const blink::WebMouseEvent*>(event));
+ // TODO(wjmaclean): Initialize latency info correctly for OOPIFs.
+ // https://crbug.com/613628
+ ui::LatencyInfo latency_info;
+ view_->ProcessMouseEvent(*static_cast<const blink::WebMouseEvent*>(event),
+ latency_info);
return;
}
if (event->type == blink::WebInputEvent::MouseWheel) {
+ // TODO(wjmaclean): Initialize latency info correctly for OOPIFs.
+ // https://crbug.com/613628
+ ui::LatencyInfo latency_info;
view_->ProcessMouseWheelEvent(
- *static_cast<const blink::WebMouseWheelEvent*>(event));
+ *static_cast<const blink::WebMouseWheelEvent*>(event), latency_info);
return;
}
}

Powered by Google App Engine
This is Rietveld 408576698