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

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, 7 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 6002fcaa6b18e3707e4430c4cdf62f085f4adbc6..5ee71a65bde5b8984d71e92bfaac6b8dc4ede822 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);
+ ui::LatencyInfo latency_info;
+ // Don't know if I add ui_latency component to the latency_info or not.
+ //I can change function signiture and pass latency to it.
tdresser 2016/05/20 17:24:53 Add a TODO and link to the bug you filed. Same th
sahel 2016/05/20 18:38:20 Done.
+ 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));
+ ui::LatencyInfo latency_info;
+ //Don't know if I should add ui_latency componenet or not.
+ //I can change function signiture and pass latency to it.
+ view_->ProcessMouseEvent(*static_cast<const blink::WebMouseEvent*>(event),
+ latency_info);
return;
}
if (event->type == blink::WebInputEvent::MouseWheel) {
+ ui::LatencyInfo latency_info;
+ //Don't know if I should add ui_latency componenet or not.
+ //I can change function signiture and pass latency to it.
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