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

Unified Diff: ash/wm/overview/window_selector_unittest.cc

Issue 2786693002: Add PointerDetails to ui::MouseEvent's constructors (Closed)
Patch Set: mouse event constructor Created 3 years, 9 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: ash/wm/overview/window_selector_unittest.cc
diff --git a/ash/wm/overview/window_selector_unittest.cc b/ash/wm/overview/window_selector_unittest.cc
index f965c35801b3bc3a9a97414080bef4ff0e0fbbe0..f3cc997a3c089b2eea30859400999a16f1db29b4 100644
--- a/ash/wm/overview/window_selector_unittest.cc
+++ b/ash/wm/overview/window_selector_unittest.cc
@@ -734,8 +734,10 @@ TEST_F(WindowSelectorTest, WindowDoesNotReceiveEvents) {
gfx::Point point1(window_bounds.x() + 10, window_bounds.y() + 10);
- ui::MouseEvent event1(ui::ET_MOUSE_PRESSED, point1, point1,
- ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
+ ui::MouseEvent event1(
+ ui::ET_MOUSE_PRESSED, point1, point1, ui::EventTimeForNow(), ui::EF_NONE,
+ ui::EF_NONE,
+ ui::PointerDetails(ui::EventPointerType::POINTER_TYPE_MOUSE));
ui::EventTarget* root_target = root_window;
ui::EventTargeter* targeter =
@@ -750,8 +752,10 @@ TEST_F(WindowSelectorTest, WindowDoesNotReceiveEvents) {
// The bounds have changed, take that into account.
gfx::Rect bounds = GetTransformedBoundsInRootWindow(window.get());
gfx::Point point2(bounds.x() + 10, bounds.y() + 10);
- ui::MouseEvent event2(ui::ET_MOUSE_PRESSED, point2, point2,
- ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
+ ui::MouseEvent event2(
+ ui::ET_MOUSE_PRESSED, point2, point2, ui::EventTimeForNow(), ui::EF_NONE,
+ ui::EF_NONE,
+ ui::PointerDetails(ui::EventPointerType::POINTER_TYPE_MOUSE));
// Now the transparent window should be intercepting this event.
EXPECT_NE(window.get(), targeter->FindTargetForEvent(root_target, &event2));

Powered by Google App Engine
This is Rietveld 408576698