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

Unified Diff: content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc

Issue 2782893002: WebMouseEvent coordinates are now fractional & private (Closed)
Patch Set: Rebased, fixed a comment in web_input_event_builders_mac.mm Created 3 years, 8 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/input/render_widget_host_latency_tracker_unittest.cc
diff --git a/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc b/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
index a161e2f0938b8dffdf03742c0d3906c14458f076..701e40262672fb8eb22085ab287d68bf1af5771b 100644
--- a/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
+++ b/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
@@ -671,8 +671,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, InputCoordinatesPopulated) {
{
auto event =
SyntheticWebMouseWheelEventBuilder::Build(0, 0, -5, 0, 0, true);
- event.x = 100;
- event.y = 200;
+ event.setPositionInWidget(100, 200);
ui::LatencyInfo latency_info;
tracker()->OnInputEvent(event, &latency_info);
EXPECT_EQ(1u, latency_info.input_coordinates_size());
@@ -682,8 +681,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, InputCoordinatesPopulated) {
{
auto event = SyntheticWebMouseEventBuilder::Build(WebInputEvent::MouseMove);
- event.x = 300;
- event.y = 400;
+ event.setPositionInWidget(300, 400);
ui::LatencyInfo latency_info;
tracker()->OnInputEvent(event, &latency_info);
EXPECT_EQ(1u, latency_info.input_coordinates_size());

Powered by Google App Engine
This is Rietveld 408576698