gfx: Make conversions from gfx::Point to PointF explicit.

ui/aura/window_event_dispatcher_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/aura/window_event_dispatcher.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/run_loop.h"
@@ skipping 98 matching lines @@
   const int kWindowWidth = 123;
   const int kWindowHeight = 45;
   gfx::Rect bounds1(100, 200, kWindowWidth, kWindowHeight);
   gfx::Rect bounds2(300, 400, kWindowWidth, kWindowHeight);
   scoped_ptr<aura::Window> window1(CreateTestWindowWithDelegate(
       delegate1.get(), -1234, bounds1, root_window()));
   scoped_ptr<aura::Window> window2(CreateTestWindowWithDelegate(
       delegate2.get(), -5678, bounds2, root_window()));
 
   // Send a mouse event to window1.
-  gfx::Point point(101, 201);
+  gfx::PointF point(101.f, 201.f);
   ui::MouseEvent event1(ui::ET_MOUSE_PRESSED, point, point,
                         ui::EventTimeForNow(), ui::EF_LEFT_MOUSE_BUTTON,
                         ui::EF_LEFT_MOUSE_BUTTON);
   DispatchEventUsingWindowDispatcher(&event1);
 
   // Event was tested for non-client area for the target window.
   EXPECT_EQ(1, delegate1->non_client_count());
   EXPECT_EQ(0, delegate2->non_client_count());
   // The non-client component test was in local coordinates.
   EXPECT_EQ(gfx::Point(1, 1), delegate1->non_client_location());
   // Mouse event was received by target window.
   EXPECT_EQ(1, delegate1->mouse_event_count());
   EXPECT_EQ(0, delegate2->mouse_event_count());
   // Event was in local coordinates.
   EXPECT_EQ(gfx::Point(1, 1), delegate1->mouse_event_location());
   // Non-client flag was set.
   EXPECT_TRUE(delegate1->mouse_event_flags() & ui::EF_IS_NON_CLIENT);
 }
 
 TEST_F(WindowEventDispatcherTest, RepostEvent) {
   // Test RepostEvent in RootWindow. It only works for Mouse Press.
   EXPECT_FALSE(Env::GetInstance()->IsMouseButtonDown());
-  gfx::Point point(10, 10);
+  gfx::PointF point(10.f, 10.f);
   ui::MouseEvent event(ui::ET_MOUSE_PRESSED, point, point,
                        ui::EventTimeForNow(), ui::EF_LEFT_MOUSE_BUTTON,
                        ui::EF_LEFT_MOUSE_BUTTON);
   host()->dispatcher()->RepostEvent(event);
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(Env::GetInstance()->IsMouseButtonDown());
 }
 
 // Check that we correctly track the state of the mouse buttons in response to
 // button press and release events.
 TEST_F(WindowEventDispatcherTest, MouseButtonState) {
   EXPECT_FALSE(Env::GetInstance()->IsMouseButtonDown());
 
-  gfx::Point location;
+  gfx::PointF location;
   scoped_ptr<ui::MouseEvent> event;
 
   // Press the left button.
   event.reset(new ui::MouseEvent(
       ui::ET_MOUSE_PRESSED, location, location, ui::EventTimeForNow(),
       ui::EF_LEFT_MOUSE_BUTTON, ui::EF_LEFT_MOUSE_BUTTON));
   DispatchEventUsingWindowDispatcher(event.get());
   EXPECT_TRUE(Env::GetInstance()->IsMouseButtonDown());
 
   // Additionally press the right.
@@ skipping 23 matching lines @@
       ui::ET_MOUSE_PRESSED, location, location, ui::EventTimeForNow(),
       ui::EF_MIDDLE_MOUSE_BUTTON, ui::EF_MIDDLE_MOUSE_BUTTON));
   DispatchEventUsingWindowDispatcher(event.get());
   EXPECT_TRUE(Env::GetInstance()->IsMouseButtonDown());
 }
 
 TEST_F(WindowEventDispatcherTest, TranslatedEvent) {
   scoped_ptr<Window> w1(test::CreateTestWindowWithDelegate(NULL, 1,
       gfx::Rect(50, 50, 100, 100), root_window()));
 
-  gfx::Point origin(100, 100);
+  gfx::PointF origin(100.f, 100.f);
   ui::MouseEvent root(ui::ET_MOUSE_PRESSED, origin, origin,
                       ui::EventTimeForNow(), 0, 0);
 
-  EXPECT_EQ("100,100", root.location().ToString());
-  EXPECT_EQ("100,100", root.root_location().ToString());
+  EXPECT_EQ(gfx::PointF(100.f, 100.f), root.location_f());
+  EXPECT_EQ(gfx::PointF(100.f, 100.1), root.root_location_f());
 
   ui::MouseEvent translated_event(
       root, static_cast<Window*>(root_window()), w1.get(),
       ui::ET_MOUSE_ENTERED, root.flags());
-  EXPECT_EQ("50,50", translated_event.location().ToString());
-  EXPECT_EQ("100,100", translated_event.root_location().ToString());
+  EXPECT_EQ(gfx::PointF(50.f, 50.f), translated_event.location_f());
+  EXPECT_EQ(gfx::PointF(100.f, 100.f), translated_event.root_location_f());
 }
 
 namespace {
 
 class TestEventClient : public client::EventClient {
  public:
   static const int kNonLockWindowId = 100;
   static const int kLockWindowId = 200;
 
   explicit TestEventClient(Window* root_window)
@@ skipping 149 matching lines @@
 
   w1->RemovePreTargetHandler(&handler);
 }
 
 // Tests that touch-events that are beyond the bounds of the root-window do get
 // propagated to the event filters correctly with the root as the target.
 TEST_F(WindowEventDispatcherTest, TouchEventsOutsideBounds) {
   ui::test::TestEventHandler handler;
   root_window()->AddPreTargetHandler(&handler);
 
-  gfx::Point position = root_window()->bounds().origin();
-  position.Offset(-10, -10);
+  auto position = gfx::PointF(root_window()->bounds().origin());
+  position.Offset(-10.f, -10.f);
   ui::TouchEvent press(
       ui::ET_TOUCH_PRESSED, position, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&press);
   EXPECT_EQ(1, handler.num_touch_events());
 
-  position = root_window()->bounds().origin();
-  position.Offset(root_window()->bounds().width() + 10,
-                  root_window()->bounds().height() + 10);
+  position = gfx::PointF(root_window()->bounds().origin());
+  position.Offset(root_window()->bounds().width() + 10.f,
+                  root_window()->bounds().height() + 10.f);
   ui::TouchEvent release(
       ui::ET_TOUCH_RELEASED, position, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&release);
   EXPECT_EQ(2, handler.num_touch_events());
   root_window()->RemovePreTargetHandler(&handler);
 }
 
 // Tests that scroll events are dispatched correctly.
 TEST_F(WindowEventDispatcherTest, ScrollEventDispatch) {
   base::TimeDelta now = ui::EventTimeForNow();
   ui::test::TestEventHandler handler;
   root_window()->AddPreTargetHandler(&handler);
 
   test::TestWindowDelegate delegate;
   scoped_ptr<Window> w1(CreateNormalWindow(1, root_window(), &delegate));
   w1->SetBounds(gfx::Rect(20, 20, 40, 40));
 
   // A scroll event on the root-window itself is dispatched.
-  ui::ScrollEvent scroll1(ui::ET_SCROLL,
-                          gfx::Point(10, 10),
-                          now,
-                          0,
-                          0, -10,
-                          0, -10,
-                          2);
+  ui::ScrollEvent scroll1(ui::ET_SCROLL, gfx::PointF(10.f, 10.f), now, 0, 0,
+                          -10, 0, -10, 2);
   DispatchEventUsingWindowDispatcher(&scroll1);
   EXPECT_EQ(1, handler.num_scroll_events());
 
   // Scroll event on a window should be dispatched properly.
-  ui::ScrollEvent scroll2(ui::ET_SCROLL,
-                          gfx::Point(25, 30),
-                          now,
-                          0,
-                          -10, 0,
-                          -10, 0,
-                          2);
+  ui::ScrollEvent scroll2(ui::ET_SCROLL, gfx::PointF(25.f, 30.f), now, 0, -10,
+                          0, -10, 0, 2);
   DispatchEventUsingWindowDispatcher(&scroll2);
   EXPECT_EQ(2, handler.num_scroll_events());
   root_window()->RemovePreTargetHandler(&handler);
 }
 
 namespace {
 
 // FilterFilter that tracks the types of events it's seen.
 class EventFilterRecorder : public ui::EventHandler {
  public:
@@ skipping 183 matching lines @@
   // Set capture on |window| generate a mouse event (that is reposted) and not
   // over |window| and verify |window| gets it (|window| gets it because it has
   // capture).
   EXPECT_FALSE(Env::GetInstance()->IsMouseButtonDown());
   EventFilterRecorder recorder;
   scoped_ptr<Window> window(CreateNormalWindow(1, root_window(), NULL));
   window->SetBounds(gfx::Rect(20, 20, 40, 30));
   window->AddPreTargetHandler(&recorder);
   window->SetCapture();
   const ui::MouseEvent press_event(
-      ui::ET_MOUSE_PRESSED, gfx::Point(), gfx::Point(), ui::EventTimeForNow(),
+      ui::ET_MOUSE_PRESSED, gfx::PointF(), gfx::PointF(), ui::EventTimeForNow(),
       ui::EF_LEFT_MOUSE_BUTTON, ui::EF_LEFT_MOUSE_BUTTON);
   host()->dispatcher()->RepostEvent(press_event);
   RunAllPendingInMessageLoop();  // Necessitated by RepostEvent().
   // Mouse moves/enters may be generated. We only care about a pressed.
   EXPECT_TRUE(EventTypesToString(recorder.events()).find("MOUSE_PRESSED") !=
               std::string::npos) << EventTypesToString(recorder.events());
 }
 
 TEST_F(WindowEventDispatcherTest, MouseMovesHeld) {
   EventFilterRecorder recorder;
   root_window()->AddPreTargetHandler(&recorder);
 
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1, gfx::Rect(0, 0, 100, 100), root_window()));
 
-  ui::MouseEvent mouse_move_event(ui::ET_MOUSE_MOVED, gfx::Point(0, 0),
-                                  gfx::Point(0, 0), ui::EventTimeForNow(), 0,
-                                  0);
+  ui::MouseEvent mouse_move_event(ui::ET_MOUSE_MOVED, gfx::PointF(0.f, 0.f),
+                                  gfx::PointF(0.f, 0.f), ui::EventTimeForNow(),
+                                  0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_move_event);
   // Discard MOUSE_ENTER.
   recorder.Reset();
 
   host()->dispatcher()->HoldPointerMoves();
 
   // Check that we don't immediately dispatch the MOUSE_DRAGGED event.
-  ui::MouseEvent mouse_dragged_event(ui::ET_MOUSE_DRAGGED, gfx::Point(0, 0),
-                                     gfx::Point(0, 0), ui::EventTimeForNow(), 0,
-                                     0);
+  ui::MouseEvent mouse_dragged_event(
+      ui::ET_MOUSE_DRAGGED, gfx::PointF(0.f, 0.f), gfx::PointF(0.f, 0.f),
+      ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event);
   EXPECT_TRUE(recorder.events().empty());
 
   // Check that we do dispatch the held MOUSE_DRAGGED event before another type
   // of event.
-  ui::MouseEvent mouse_pressed_event(ui::ET_MOUSE_PRESSED, gfx::Point(0, 0),
-                                     gfx::Point(0, 0), ui::EventTimeForNow(), 0,
-                                     0);
+  ui::MouseEvent mouse_pressed_event(
+      ui::ET_MOUSE_PRESSED, gfx::PointF(0.f, 0.f), gfx::PointF(0.f, 0.f),
+      ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_pressed_event);
   EXPECT_EQ("MOUSE_DRAGGED MOUSE_PRESSED",
             EventTypesToString(recorder.events()));
   recorder.Reset();
 
   // Check that we coalesce held MOUSE_DRAGGED events. Note that here (and
   // elsewhere in this test) we re-define each event prior to dispatch so that
   // it has the correct state (phase, handled, target, etc.).
   mouse_dragged_event =
-      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::Point(0, 0), gfx::Point(0, 0),
-                     ui::EventTimeForNow(), 0, 0);
-  ui::MouseEvent mouse_dragged_event2(ui::ET_MOUSE_DRAGGED, gfx::Point(10, 10),
-                                      gfx::Point(10, 10), ui::EventTimeForNow(),
-                                      0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::PointF(0.f, 0.f),
+                     gfx::PointF(0.f, 0.f), ui::EventTimeForNow(), 0, 0);
+  ui::MouseEvent mouse_dragged_event2(
+      ui::ET_MOUSE_DRAGGED, gfx::PointF(10.f, 10.f), gfx::PointF(10.f, 10.f),
+      ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event2);
   EXPECT_TRUE(recorder.events().empty());
   mouse_pressed_event =
-      ui::MouseEvent(ui::ET_MOUSE_PRESSED, gfx::Point(0, 0), gfx::Point(0, 0),
-                     ui::EventTimeForNow(), 0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_PRESSED, gfx::PointF(0.f, 0.f),
+                     gfx::PointF(0.f, 0.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_pressed_event);
   EXPECT_EQ("MOUSE_DRAGGED MOUSE_PRESSED",
             EventTypesToString(recorder.events()));
   recorder.Reset();
 
   // Check that on ReleasePointerMoves, held events are not dispatched
   // immediately, but posted instead.
   mouse_dragged_event =
-      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::Point(0, 0), gfx::Point(0, 0),
-                     ui::EventTimeForNow(), 0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::PointF(0.f, 0.f),
+                     gfx::PointF(0.f, 0.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event);
   host()->dispatcher()->ReleasePointerMoves();
   EXPECT_TRUE(recorder.events().empty());
   RunAllPendingInMessageLoop();
   EXPECT_EQ("MOUSE_DRAGGED", EventTypesToString(recorder.events()));
   recorder.Reset();
 
   // However if another message comes in before the dispatch of the posted
   // event, check that the posted event is dispatched before this new event.
   host()->dispatcher()->HoldPointerMoves();
   mouse_dragged_event =
-      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::Point(0, 0), gfx::Point(0, 0),
-                     ui::EventTimeForNow(), 0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::PointF(0.f, 0.f),
+                     gfx::PointF(0.f, 0.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event);
   host()->dispatcher()->ReleasePointerMoves();
   mouse_pressed_event =
-      ui::MouseEvent(ui::ET_MOUSE_PRESSED, gfx::Point(0, 0), gfx::Point(0, 0),
-                     ui::EventTimeForNow(), 0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_PRESSED, gfx::PointF(0.f, 0.f),
+                     gfx::PointF(0.f, 0.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_pressed_event);
   EXPECT_EQ("MOUSE_DRAGGED MOUSE_PRESSED",
             EventTypesToString(recorder.events()));
   recorder.Reset();
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(recorder.events().empty());
 
   // Check that if the other message is another MOUSE_DRAGGED, we still coalesce
   // them.
   host()->dispatcher()->HoldPointerMoves();
   mouse_dragged_event =
-      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::Point(0, 0), gfx::Point(0, 0),
-                     ui::EventTimeForNow(), 0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::PointF(0.f, 0.f),
+                     gfx::PointF(0.f, 0.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event);
   host()->dispatcher()->ReleasePointerMoves();
   mouse_dragged_event2 =
-      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::Point(10, 10),
-                     gfx::Point(10, 10), ui::EventTimeForNow(), 0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::PointF(10.f, 10.f),
+                     gfx::PointF(10.f, 10.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event2);
   EXPECT_EQ("MOUSE_DRAGGED", EventTypesToString(recorder.events()));
   recorder.Reset();
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(recorder.events().empty());
 
   // Check that synthetic mouse move event has a right location when issued
   // while holding pointer moves.
   mouse_dragged_event =
-      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::Point(0, 0), gfx::Point(0, 0),
-                     ui::EventTimeForNow(), 0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::PointF(0.f, 0.f),
+                     gfx::PointF(0.f, 0.f), ui::EventTimeForNow(), 0, 0);
   mouse_dragged_event2 =
-      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::Point(10, 10),
-                     gfx::Point(10, 10), ui::EventTimeForNow(), 0, 0);
-  ui::MouseEvent mouse_dragged_event3(ui::ET_MOUSE_DRAGGED, gfx::Point(28, 28),
-                                      gfx::Point(28, 28), ui::EventTimeForNow(),
-                                      0, 0);
+      ui::MouseEvent(ui::ET_MOUSE_DRAGGED, gfx::PointF(10.f, 10.f),
+                     gfx::PointF(10.f, 10.f), ui::EventTimeForNow(), 0, 0);
+  ui::MouseEvent mouse_dragged_event3(
+      ui::ET_MOUSE_DRAGGED, gfx::PointF(28.f, 28.f), gfx::PointF(28.f, 28.f),
+      ui::EventTimeForNow(), 0, 0);
   host()->dispatcher()->HoldPointerMoves();
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event);
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event2);
   window->SetBounds(gfx::Rect(15, 15, 80, 80));
   DispatchEventUsingWindowDispatcher(&mouse_dragged_event3);
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(recorder.events().empty());
   host()->dispatcher()->ReleasePointerMoves();
   RunAllPendingInMessageLoop();
   EXPECT_EQ("MOUSE_MOVED", EventTypesToString(recorder.events()));
   EXPECT_EQ(gfx::Point(13, 13), recorder.mouse_location(0));
   recorder.Reset();
   root_window()->RemovePreTargetHandler(&recorder);
 }
 
 TEST_F(WindowEventDispatcherTest, TouchMovesHeld) {
   EventFilterRecorder recorder;
   root_window()->AddPreTargetHandler(&recorder);
 
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1, gfx::Rect(50, 50, 100, 100), root_window()));
 
   // Starting the touch and throwing out the first few events, since the system
   // is going to generate synthetic mouse events that are not relevant to the
   // test.
   ui::TouchEvent touch_pressed_event(
-      ui::ET_TOUCH_PRESSED, gfx::Point(10, 10), 0, ui::EventTimeForNow());
+      ui::ET_TOUCH_PRESSED, gfx::PointF(10.f, 10.f), 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&touch_pressed_event);
   recorder.WaitUntilReceivedEvent(ui::ET_GESTURE_SHOW_PRESS);
   recorder.Reset();
 
   host()->dispatcher()->HoldPointerMoves();
 
   // Check that we don't immediately dispatch the TOUCH_MOVED event.
-  ui::TouchEvent touch_moved_event(
-      ui::ET_TOUCH_MOVED, gfx::Point(10, 10), 0, ui::EventTimeForNow());
-  ui::TouchEvent touch_moved_event2(
-      ui::ET_TOUCH_MOVED, gfx::Point(11, 10), 0, ui::EventTimeForNow());
-  ui::TouchEvent touch_moved_event3(
-      ui::ET_TOUCH_MOVED, gfx::Point(12, 10), 0, ui::EventTimeForNow());
+  ui::TouchEvent touch_moved_event(ui::ET_TOUCH_MOVED, gfx::PointF(10.f, 10.f),
+                                   0, ui::EventTimeForNow());
+  ui::TouchEvent touch_moved_event2(ui::ET_TOUCH_MOVED, gfx::PointF(11.f, 10.f),
+                                    0, ui::EventTimeForNow());
+  ui::TouchEvent touch_moved_event3(ui::ET_TOUCH_MOVED, gfx::PointF(12.f, 10.f),
+                                    0, ui::EventTimeForNow());
 
   DispatchEventUsingWindowDispatcher(&touch_moved_event);
   EXPECT_TRUE(recorder.events().empty());
 
   // Check that on ReleasePointerMoves, held events are not dispatched
   // immediately, but posted instead.
   DispatchEventUsingWindowDispatcher(&touch_moved_event2);
   host()->dispatcher()->ReleasePointerMoves();
   EXPECT_TRUE(recorder.events().empty());
 
   RunAllPendingInMessageLoop();
   EXPECT_EQ("TOUCH_MOVED", EventTypesToString(recorder.events()));
   recorder.Reset();
 
   // If another touch event occurs then the held touch should be dispatched
   // immediately before it.
   ui::TouchEvent touch_released_event(
-      ui::ET_TOUCH_RELEASED, gfx::Point(10, 10), 0, ui::EventTimeForNow());
+      ui::ET_TOUCH_RELEASED, gfx::PointF(10.f, 10.f), 0, ui::EventTimeForNow());
   recorder.Reset();
   host()->dispatcher()->HoldPointerMoves();
   DispatchEventUsingWindowDispatcher(&touch_moved_event3);
   DispatchEventUsingWindowDispatcher(&touch_released_event);
   EXPECT_EQ("TOUCH_MOVED TOUCH_RELEASED GESTURE_TAP GESTURE_END",
             EventTypesToString(recorder.events()));
   recorder.Reset();
   host()->dispatcher()->ReleasePointerMoves();
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(recorder.events().empty());
@@ skipping 10 matching lines @@
   scoped_ptr<aura::Window> window_first(CreateTestWindowWithDelegate(
       &delegate, 1, gfx::Rect(20, 10, 10, 20), root_window()));
   window_first->Show();
   window_first->AddPreTargetHandler(&recorder_first);
 
   scoped_ptr<aura::Window> window_second(CreateTestWindowWithDelegate(
       &delegate, 2, gfx::Rect(20, 30, 10, 20), root_window()));
   window_second->Show();
   window_second->AddPreTargetHandler(&recorder_second);
 
-  const gfx::Point event_location(22, 33);
+  const gfx::PointF event_location(22.f, 33.f);
   ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, event_location, event_location,
                        ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse);
 
   EXPECT_TRUE(recorder_first.events().empty());
   EXPECT_EQ("MOUSE_ENTERED MOUSE_MOVED",
             EventTypesToString(recorder_second.events()));
   ASSERT_EQ(2u, recorder_second.mouse_locations().size());
   EXPECT_EQ(gfx::Point(2, 3).ToString(),
             recorder_second.mouse_locations()[0].ToString());
 }
 
 // Tests that a mouse exit is dispatched to the last mouse location when
 // the window is hiddden.
 TEST_F(WindowEventDispatcherTest, DispatchMouseExitWhenHidingWindow) {
   EventFilterRecorder recorder;
 
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1, gfx::Rect(10, 10, 50, 50), root_window()));
   window->Show();
   window->AddPreTargetHandler(&recorder);
 
   // Dispatch a mouse move event into the window.
-  const gfx::Point event_location(22, 33);
+  const gfx::PointF event_location(22.f, 33.f);
   ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, event_location, event_location,
                        ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse);
   EXPECT_FALSE(recorder.events().empty());
   recorder.Reset();
 
   // Hide the window and verify a mouse exit event's location.
   window->Hide();
   EXPECT_FALSE(recorder.events().empty());
   EXPECT_EQ("MOUSE_EXITED", EventTypesToString(recorder.events()));
@@ skipping 75 matching lines @@
             recorder.touch_locations()[1].ToString());
   root_window()->RemovePreTargetHandler(&recorder);
 }
 
 // Tests that a 'held' touch-event does contribute to gesture event when it is
 // dispatched.
 TEST_F(WindowEventDispatcherTest, HeldTouchMoveContributesToGesture) {
   EventFilterRecorder recorder;
   root_window()->AddPreTargetHandler(&recorder);
 
-  const gfx::Point location(20, 20);
+  const gfx::PointF location(20.f, 20.f);
   ui::TouchEvent press(
       ui::ET_TOUCH_PRESSED, location, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&press);
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_TOUCH_PRESSED));
   recorder.Reset();
 
   host()->dispatcher()->HoldPointerMoves();
 
   ui::TouchEvent move(ui::ET_TOUCH_MOVED,
-                      location + gfx::Vector2d(100, 100),
-                      0,
+                      location + gfx::Vector2dF(100.f, 100.f), 0,
                       ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&move);
   EXPECT_FALSE(recorder.HasReceivedEvent(ui::ET_TOUCH_MOVED));
   EXPECT_FALSE(recorder.HasReceivedEvent(ui::ET_GESTURE_SCROLL_BEGIN));
   recorder.Reset();
 
   host()->dispatcher()->ReleasePointerMoves();
   EXPECT_FALSE(recorder.HasReceivedEvent(ui::ET_TOUCH_MOVED));
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_TOUCH_MOVED));
@@ skipping 11 matching lines @@
 
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1234, gfx::Rect(5, 5, 100, 100), root_window()));
   window->Show();
   window->SetCapture();
 
   test::TestCursorClient cursor_client(root_window());
 
   // Dispatch a non-synthetic mouse event when mouse events are enabled.
-  ui::MouseEvent mouse1(ui::ET_MOUSE_MOVED, gfx::Point(10, 10),
-                        gfx::Point(10, 10), ui::EventTimeForNow(), 0, 0);
+  ui::MouseEvent mouse1(ui::ET_MOUSE_MOVED, gfx::PointF(10.f, 10.f),
+                        gfx::PointF(10.f, 10.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse1);
   EXPECT_FALSE(recorder.events().empty());
   recorder.Reset();
 
   // Dispatch a synthetic mouse event when mouse events are enabled.
-  ui::MouseEvent mouse2(ui::ET_MOUSE_MOVED, gfx::Point(10, 10),
-                        gfx::Point(10, 10), ui::EventTimeForNow(),
+  ui::MouseEvent mouse2(ui::ET_MOUSE_MOVED, gfx::PointF(10.f, 10.f),
+                        gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
                         ui::EF_IS_SYNTHESIZED, 0);
   DispatchEventUsingWindowDispatcher(&mouse2);
   EXPECT_FALSE(recorder.events().empty());
   recorder.Reset();
 
   // Dispatch a synthetic mouse event when mouse events are disabled.
   cursor_client.DisableMouseEvents();
   DispatchEventUsingWindowDispatcher(&mouse2);
   EXPECT_TRUE(recorder.events().empty());
   root_window()->RemovePreTargetHandler(&recorder);
 }
 
 // Tests that a mouse-move event is not synthesized when a mouse-button is down.
 TEST_F(WindowEventDispatcherTest, DoNotSynthesizeWhileButtonDown) {
   EventFilterRecorder recorder;
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1234, gfx::Rect(5, 5, 100, 100), root_window()));
   window->Show();
 
   window->AddPreTargetHandler(&recorder);
   // Dispatch a non-synthetic mouse event when mouse events are enabled.
-  ui::MouseEvent mouse1(ui::ET_MOUSE_PRESSED, gfx::Point(10, 10),
-                        gfx::Point(10, 10), ui::EventTimeForNow(),
+  ui::MouseEvent mouse1(ui::ET_MOUSE_PRESSED, gfx::PointF(10.f, 10.f),
+                        gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
                         ui::EF_LEFT_MOUSE_BUTTON, ui::EF_LEFT_MOUSE_BUTTON);
   DispatchEventUsingWindowDispatcher(&mouse1);
   ASSERT_EQ(1u, recorder.events().size());
   EXPECT_EQ(ui::ET_MOUSE_PRESSED, recorder.events()[0]);
   window->RemovePreTargetHandler(&recorder);
   recorder.Reset();
 
   // Move |window| away from underneath the cursor.
   root_window()->AddPreTargetHandler(&recorder);
   window->SetBounds(gfx::Rect(30, 30, 100, 100));
@@ skipping 18 matching lines @@
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1234, gfx::Rect(5, 5, 100, 100), root_window()));
   window->Show();
   window->SetCapture();
 
   EventFilterRecorder recorder;
   window->AddPreTargetHandler(&recorder);
 
   // Dispatch a non-synthetic mouse event to place cursor inside window bounds.
-  ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, gfx::Point(10, 10),
-                       gfx::Point(10, 10), ui::EventTimeForNow(), 0, 0);
+  ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, gfx::PointF(10.f, 10.f),
+                       gfx::PointF(10.f, 10.f), ui::EventTimeForNow(), 0, 0);
   DispatchEventUsingWindowDispatcher(&mouse);
   EXPECT_FALSE(recorder.events().empty());
   recorder.Reset();
 
   // Update the window bounds so that cursor is now outside the window.
   // This should trigger a synthetic MOVED event.
   gfx::Rect bounds1(20, 20, 100, 100);
   window->SetBounds(bounds1);
   RunAllPendingInMessageLoop();
   ASSERT_FALSE(recorder.events().empty());
@@ skipping 32 matching lines @@
   root_window()->AddPreTargetHandler(&recorder);
 
   test::TestWindowDelegate delegate;
   gfx::Point window_origin(7, 18);
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1234, gfx::Rect(window_origin, gfx::Size(100, 100)),
       root_window()));
   window->Show();
 
   // Dispatch a mouse move event into the window.
-  gfx::Point mouse_location(gfx::Point(15, 25));
+  gfx::PointF mouse_location(15., 25.f);
   ui::MouseEvent mouse1(ui::ET_MOUSE_MOVED, mouse_location, mouse_location,
                         ui::EventTimeForNow(), 0, 0);
   EXPECT_TRUE(recorder.events().empty());
   DispatchEventUsingWindowDispatcher(&mouse1);
   EXPECT_FALSE(recorder.events().empty());
   recorder.Reset();
 
   // Hide the cursor and verify a mouse exit was dispatched.
   host()->OnCursorVisibilityChanged(false);
   EXPECT_FALSE(recorder.events().empty());
@@ skipping 16 matching lines @@
   root_window()->AddPreTargetHandler(&recorder);
 
   test::TestWindowDelegate delegate;
   gfx::Point window_origin(7, 18);
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1234, gfx::Rect(window_origin, gfx::Size(100, 100)),
       root_window()));
   window->Show();
 
   // Dispatch a mouse move event into the window.
-  gfx::Point mouse_location(gfx::Point(15, 25));
+  gfx::PointF mouse_location(15.f, 25.f);
   ui::MouseEvent mouse1(ui::ET_MOUSE_MOVED, mouse_location, mouse_location,
                         ui::EventTimeForNow(), 0, 0);
   EXPECT_TRUE(recorder.events().empty());
   DispatchEventUsingWindowDispatcher(&mouse1);
   EXPECT_FALSE(recorder.events().empty());
   recorder.Reset();
 
   test::TestCursorClient cursor_client(root_window());
   cursor_client.DisableMouseEvents();
 
   gfx::Point mouse_exit_location(gfx::Point(150, 150));
-  ui::MouseEvent mouse2(ui::ET_MOUSE_EXITED, gfx::Point(150, 150),
-                        gfx::Point(150, 150), ui::EventTimeForNow(),
+  ui::MouseEvent mouse2(ui::ET_MOUSE_EXITED, gfx::PointF(150.f, 150.f),
+                        gfx::PointF(150.f, 150.f), ui::EventTimeForNow(),
                         ui::EF_IS_SYNTHESIZED, 0);
   DispatchEventUsingWindowDispatcher(&mouse2);
 
   EXPECT_FALSE(recorder.events().empty());
   // We get the mouse exited event twice in our filter. Once during the
   // predispatch phase and during the actual dispatch.
   EXPECT_EQ("MOUSE_EXITED MOUSE_EXITED", EventTypesToString(recorder.events()));
 
   // Verify the mouse exit was dispatched at the correct location
   // (in the correct coordinate space).
@@ skipping 463 matching lines @@
       : root_(root),
         mouse_event_count_(0) {}
   ~DontResetHeldEventWindowDelegate() override {}
 
   int mouse_event_count() const { return mouse_event_count_; }
 
   // TestWindowDelegate:
   void OnMouseEvent(ui::MouseEvent* event) override {
     if ((event->flags() & ui::EF_SHIFT_DOWN) != 0 &&
         mouse_event_count_++ == 0) {
-      ui::MouseEvent mouse_event(ui::ET_MOUSE_PRESSED, gfx::Point(10, 10),
-                                 gfx::Point(10, 10), ui::EventTimeForNow(),
+      ui::MouseEvent mouse_event(ui::ET_MOUSE_PRESSED, gfx::PointF(10.f, 10.f),
+                                 gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
                                  ui::EF_SHIFT_DOWN, 0);
       root_->GetHost()->dispatcher()->RepostEvent(mouse_event);
     }
   }
 
  private:
   Window* root_;
   int mouse_event_count_;
 
   DISALLOW_COPY_AND_ASSIGN(DontResetHeldEventWindowDelegate);
 };
 
 }  // namespace
 
 // Verifies RootWindow doesn't reset |RootWindow::held_repostable_event_| after
 // dispatching. This is done by using DontResetHeldEventWindowDelegate, which
 // tracks the number of events with ui::EF_SHIFT_DOWN set (all reposted events
 // have EF_SHIFT_DOWN). When the first event is seen RepostEvent() is used to
 // schedule another reposted event.
 TEST_F(WindowEventDispatcherTest, DontResetHeldEvent) {
   DontResetHeldEventWindowDelegate delegate(root_window());
   scoped_ptr<Window> w1(CreateNormalWindow(1, root_window(), &delegate));
   w1->SetBounds(gfx::Rect(0, 0, 40, 40));
-  ui::MouseEvent pressed(ui::ET_MOUSE_PRESSED, gfx::Point(10, 10),
-                         gfx::Point(10, 10), ui::EventTimeForNow(),
+  ui::MouseEvent pressed(ui::ET_MOUSE_PRESSED, gfx::PointF(10.f, 10.f),
+                         gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
                          ui::EF_SHIFT_DOWN, 0);
   root_window()->GetHost()->dispatcher()->RepostEvent(pressed);
-  ui::MouseEvent pressed2(ui::ET_MOUSE_PRESSED, gfx::Point(10, 10),
-                          gfx::Point(10, 10), ui::EventTimeForNow(), 0, 0);
+  ui::MouseEvent pressed2(ui::ET_MOUSE_PRESSED, gfx::PointF(10.f, 10.f),
+                          gfx::PointF(10.f, 10.f), ui::EventTimeForNow(), 0, 0);
   // Dispatch an event to flush event scheduled by way of RepostEvent().
   DispatchEventUsingWindowDispatcher(&pressed2);
   // Delegate should have seen reposted event (identified by way of
   // EF_SHIFT_DOWN). Dispatch another event to flush the second
   // RepostedEvent().
   EXPECT_EQ(1, delegate.mouse_event_count());
   DispatchEventUsingWindowDispatcher(&pressed2);
   EXPECT_EQ(2, delegate.mouse_event_count());
 }
 
@@ skipping 35 matching lines @@
 // Verifies if a WindowTreeHost is deleted from dispatching a held mouse event
 // we don't crash.
 TEST_F(WindowEventDispatcherTest, DeleteHostFromHeldMouseEvent) {
   // Should be deleted by |delegate|.
   WindowTreeHost* h2 = WindowTreeHost::Create(gfx::Rect(0, 0, 100, 100));
   h2->InitHost();
   DeleteHostFromHeldMouseEventDelegate delegate(h2);
   // Owned by |h2|.
   Window* w1 = CreateNormalWindow(1, h2->window(), &delegate);
   w1->SetBounds(gfx::Rect(0, 0, 40, 40));
-  ui::MouseEvent pressed(ui::ET_MOUSE_PRESSED, gfx::Point(10, 10),
-                         gfx::Point(10, 10), ui::EventTimeForNow(),
+  ui::MouseEvent pressed(ui::ET_MOUSE_PRESSED, gfx::PointF(10.f, 10.f),
+                         gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
                          ui::EF_SHIFT_DOWN, 0);
   h2->dispatcher()->RepostEvent(pressed);
   // RunAllPendingInMessageLoop() to make sure the |pressed| is run.
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(delegate.got_mouse_event());
   EXPECT_TRUE(delegate.got_destroy());
 }
 
 TEST_F(WindowEventDispatcherTest, WindowHideCancelsActiveTouches) {
   EventFilterRecorder recorder;
   root_window()->AddPreTargetHandler(&recorder);
 
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1, gfx::Rect(0, 0, 100, 100), root_window()));
 
-  gfx::Point position1 = root_window()->bounds().origin();
+  auto position1 = gfx::PointF(root_window()->bounds().origin());
   ui::TouchEvent press(
       ui::ET_TOUCH_PRESSED, position1, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&press);
 
   EXPECT_EQ("TOUCH_PRESSED GESTURE_BEGIN GESTURE_TAP_DOWN",
             EventTypesToString(recorder.GetAndResetEvents()));
 
   window->Hide();
 
   EXPECT_EQ(ui::ET_TOUCH_CANCELLED, recorder.events()[0]);
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_GESTURE_TAP_CANCEL));
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_GESTURE_END));
   EXPECT_EQ(3U, recorder.events().size());
   root_window()->RemovePreTargetHandler(&recorder);
 }
 
 TEST_F(WindowEventDispatcherTest, WindowHideCancelsActiveGestures) {
   EventFilterRecorder recorder;
   root_window()->AddPreTargetHandler(&recorder);
 
   test::TestWindowDelegate delegate;
   scoped_ptr<aura::Window> window(CreateTestWindowWithDelegate(
       &delegate, 1, gfx::Rect(0, 0, 100, 100), root_window()));
 
-  gfx::Point position1 = root_window()->bounds().origin();
-  gfx::Point position2 = root_window()->bounds().CenterPoint();
+  auto position1 = gfx::PointF(root_window()->bounds().origin());
+  auto position2 = gfx::PointF(root_window()->bounds().CenterPoint());
   ui::TouchEvent press(
       ui::ET_TOUCH_PRESSED, position1, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&press);
 
   ui::TouchEvent move(
       ui::ET_TOUCH_MOVED, position2, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&move);
 
   ui::TouchEvent press2(
       ui::ET_TOUCH_PRESSED, position1, 1, ui::EventTimeForNow());
@@ skipping 37 matching lines @@
   EventFilterRecorder recorder2;
   scoped_ptr<Window> window1(CreateNormalWindow(1, root_window(), NULL));
   window1->SetBounds(gfx::Rect(0, 0, 40, 40));
 
   scoped_ptr<Window> window2(CreateNormalWindow(2, root_window(), NULL));
   window2->SetBounds(gfx::Rect(40, 0, 40, 40));
 
   window1->AddPreTargetHandler(&recorder1);
   window2->AddPreTargetHandler(&recorder2);
 
-  gfx::Point position = window1->bounds().origin();
+  auto position = gfx::PointF(window1->bounds().origin());
   ui::TouchEvent press(
       ui::ET_TOUCH_PRESSED, position, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&press);
 
-  gfx::Point position2 = window1->bounds().CenterPoint();
+  auto position2 = gfx::PointF(window1->bounds().CenterPoint());
   ui::TouchEvent move(
       ui::ET_TOUCH_MOVED, position2, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&move);
 
   window2->SetCapture();
 
   EXPECT_EQ("TOUCH_PRESSED GESTURE_BEGIN GESTURE_TAP_DOWN TOUCH_MOVED "
             "GESTURE_TAP_CANCEL GESTURE_SCROLL_BEGIN GESTURE_SCROLL_UPDATE "
             "TOUCH_CANCELLED GESTURE_SCROLL_END GESTURE_END",
             EventTypesToString(recorder1.events()));
@@ skipping 77 matching lines @@
 
   void RunTest() {
     // Reset any event the window may have received when bringing up the window
     // (e.g. mouse-move events if the mouse cursor is over the window).
     handler_.Reset();
 
     // Start a nested message-loop, post an event to be dispatched, and then
     // terminate the message-loop. When the message-loop unwinds and gets back,
     // the reposted event should not have fired.
     scoped_ptr<ui::MouseEvent> mouse(new ui::MouseEvent(
-        ui::ET_MOUSE_PRESSED, gfx::Point(10, 10), gfx::Point(10, 10),
+        ui::ET_MOUSE_PRESSED, gfx::PointF(10.f, 10.f), gfx::PointF(10.f, 10.f),
         ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE));
     message_loop()->PostTask(
         FROM_HERE,
         base::Bind(&WindowEventDispatcherTestWithMessageLoop::RepostEventHelper,
                    host()->dispatcher(),
                    base::Passed(&mouse)));
     message_loop()->PostTask(FROM_HERE, message_loop()->QuitClosure());
 
     base::MessageLoop::ScopedNestableTaskAllower allow(message_loop());
     base::RunLoop loop;
@@ skipping 65 matching lines @@
   scoped_ptr<aura::Window> child(test::CreateTestWindowWithDelegate(&delegate,
       1234, gfx::Rect(20, 20, 100, 100), root_window()));
   child->Show();
 
   ui::test::TestEventHandler handler_child;
   ui::test::TestEventHandler handler_root;
   root_window()->AddPreTargetHandler(&handler_root);
   child->AddPreTargetHandler(&handler_child);
 
   {
-    ui::MouseEvent move(ui::ET_MOUSE_MOVED, gfx::Point(30, 30),
-                        gfx::Point(30, 30), ui::EventTimeForNow(), ui::EF_NONE,
-                        ui::EF_NONE);
+    ui::MouseEvent move(ui::ET_MOUSE_MOVED, gfx::PointF(30.f, 30.f),
+                        gfx::PointF(30.f, 30.f), ui::EventTimeForNow(),
+                        ui::EF_NONE, ui::EF_NONE);
     DispatchEventUsingWindowDispatcher(&move);
     EXPECT_EQ(0, handler_child.num_mouse_events());
     EXPECT_EQ(1, handler_root.num_mouse_events());
   }
 
   {
-    ui::MouseEvent move(ui::ET_MOUSE_MOVED, gfx::Point(50, 50),
-                        gfx::Point(50, 50), ui::EventTimeForNow(), ui::EF_NONE,
-                        ui::EF_NONE);
+    ui::MouseEvent move(ui::ET_MOUSE_MOVED, gfx::PointF(50.f, 50.f),
+                        gfx::PointF(50.f, 50.f), ui::EventTimeForNow(),
+                        ui::EF_NONE, ui::EF_NONE);
     DispatchEventUsingWindowDispatcher(&move);
     // The child receives an ENTER, and a MOVED event.
     EXPECT_EQ(2, handler_child.num_mouse_events());
     // The root receives both the ENTER and the MOVED events dispatched to
     // |child|, as well as an EXIT event.
     EXPECT_EQ(3, handler_root.num_mouse_events());
   }
 
   child->RemovePreTargetHandler(&handler_child);
   root_window()->RemovePreTargetHandler(&handler_root);
@@ skipping 68 matching lines @@
 
 // Tests that if dispatching a 'held' event triggers a nested message loop, then
 // the events that are dispatched from the nested message loop are transformed
 // correctly.
 TEST_F(WindowEventDispatcherTestInHighDPI,
        EventsTransformedInRepostedEventTriggeredNestedLoop) {
   scoped_ptr<Window> window(CreateNormalWindow(1, root_window(), NULL));
   // Make sure the window is visible.
   RunAllPendingInMessageLoop();
 
-  ui::MouseEvent mouse_move(ui::ET_MOUSE_MOVED, gfx::Point(80, 80),
-                            gfx::Point(80, 80), ui::EventTimeForNow(),
+  ui::MouseEvent mouse_move(ui::ET_MOUSE_MOVED, gfx::PointF(80.f, 80.f),
+                            gfx::PointF(80.f, 80.f), ui::EventTimeForNow(),
                             ui::EF_NONE, ui::EF_NONE);
   const base::Closure callback_on_right_click = base::Bind(
       base::IgnoreResult(&WindowEventDispatcherTestInHighDPI::DispatchEvent),
       base::Unretained(this), base::Unretained(&mouse_move));
   TriggerNestedLoopOnRightMousePress handler(callback_on_right_click);
   window->AddPreTargetHandler(&handler);
 
   scoped_ptr<ui::MouseEvent> mouse(
-      new ui::MouseEvent(ui::ET_MOUSE_PRESSED, gfx::Point(10, 10),
-                         gfx::Point(10, 10), ui::EventTimeForNow(),
+      new ui::MouseEvent(ui::ET_MOUSE_PRESSED, gfx::PointF(10.f, 10.f),
+                         gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
                          ui::EF_RIGHT_MOUSE_BUTTON, ui::EF_RIGHT_MOUSE_BUTTON));
   host()->dispatcher()->RepostEvent(*mouse);
   EXPECT_EQ(0, handler.num_mouse_events());
 
   base::RunLoop run_loop;
   run_loop.RunUntilIdle();
   // The window should receive the mouse-press and the mouse-move events.
   EXPECT_EQ(2, handler.num_mouse_events());
   // The mouse-move event location should be transformed because of the DSF
   // before it reaches the window.
@@ skipping 21 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(SelfDestructDelegate);
 };
 
 TEST_F(WindowEventDispatcherTest, SynthesizedLocatedEvent) {
   ui::test::EventGenerator generator(root_window());
   generator.MoveMouseTo(10, 10);
   EXPECT_EQ("10,10",
             Env::GetInstance()->last_mouse_location().ToString());
 
   // Synthesized event should not update the mouse location.
-  ui::MouseEvent mouseev(ui::ET_MOUSE_MOVED, gfx::Point(), gfx::Point(),
+  ui::MouseEvent mouseev(ui::ET_MOUSE_MOVED, gfx::PointF(), gfx::PointF(),
                          ui::EventTimeForNow(), ui::EF_IS_SYNTHESIZED, 0);
   generator.Dispatch(&mouseev);
   EXPECT_EQ("10,10",
             Env::GetInstance()->last_mouse_location().ToString());
 
   generator.MoveMouseTo(0, 0);
   EXPECT_EQ("0,0",
             Env::GetInstance()->last_mouse_location().ToString());
 
   // Make sure the location gets updated when a syntheiszed enter
@@ skipping 74 matching lines @@
   explicit DispatchEventHandler(Window* target)
       : target_(target),
         dispatched_(false) {}
   ~DispatchEventHandler() override {}
 
   bool dispatched() const { return dispatched_; }
  private:
   // ui::EventHandler:
   void OnMouseEvent(ui::MouseEvent* mouse) override {
     if (mouse->type() == ui::ET_MOUSE_MOVED) {
-      ui::MouseEvent move(ui::ET_MOUSE_MOVED, target_->bounds().CenterPoint(),
-                          target_->bounds().CenterPoint(),
+      ui::MouseEvent move(ui::ET_MOUSE_MOVED,
+                          gfx::PointF(target_->bounds().CenterPoint()),
+                          gfx::PointF(target_->bounds().CenterPoint()),
                           ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
       ui::EventDispatchDetails details =
           target_->GetHost()->dispatcher()->OnEventFromSource(&move);
       ASSERT_FALSE(details.dispatcher_destroyed);
       EXPECT_FALSE(details.target_destroyed);
       EXPECT_EQ(target_, move.target());
       dispatched_ = true;
     }
     ui::EventHandler::OnMouseEvent(mouse);
   }
@@ skipping 52 matching lines @@
   // when |second| receives an event.
   MoveWindowHandler move_window(first.get(), second_root);
   second->AddPreTargetHandler(&move_window);
 
   // Some sanity checks: |first| is inside |root_window()|'s tree.
   EXPECT_EQ(root_window(), first->GetRootWindow());
   // The two root windows are different.
   EXPECT_NE(root_window(), second_root);
 
   // Dispatch an event to |first|.
-  ui::MouseEvent move(ui::ET_MOUSE_MOVED, first->bounds().CenterPoint(),
-                      first->bounds().CenterPoint(), ui::EventTimeForNow(),
-                      ui::EF_NONE, ui::EF_NONE);
+  ui::MouseEvent move(ui::ET_MOUSE_MOVED,
+                      gfx::PointF(first->bounds().CenterPoint()),
+                      gfx::PointF(first->bounds().CenterPoint()),
+                      ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
   ui::EventDispatchDetails details =
       host()->dispatcher()->OnEventFromSource(&move);
   ASSERT_FALSE(details.dispatcher_destroyed);
   EXPECT_TRUE(details.target_destroyed);
   EXPECT_EQ(first.get(), move.target());
   EXPECT_TRUE(dispatch_event.dispatched());
   EXPECT_EQ(second_root, first->GetRootWindow());
 
   first->RemovePreTargetHandler(&dispatch_event);
   second->RemovePreTargetHandler(&move_window);
@@ skipping 75 matching lines @@
   window_second->SetCapture();
   EXPECT_EQ(window_second.get(),
             client::GetCaptureWindow(root_window()));
 
   // Send an event to the first host. Make sure it goes to |window_second| in
   // |second_host| instead (since it has capture).
   EventFilterRecorder recorder_first;
   window_first->AddPreTargetHandler(&recorder_first);
   EventFilterRecorder recorder_second;
   window_second->AddPreTargetHandler(&recorder_second);
-  const gfx::Point event_location(25, 15);
+  const gfx::PointF event_location(25.f, 15.f);
   ui::MouseEvent mouse(ui::ET_MOUSE_PRESSED, event_location, event_location,
                        ui::EventTimeForNow(), ui::EF_LEFT_MOUSE_BUTTON,
                        ui::EF_LEFT_MOUSE_BUTTON);
   DispatchEventUsingWindowDispatcher(&mouse);
   EXPECT_TRUE(recorder_first.events().empty());
   ASSERT_EQ(1u, recorder_second.events().size());
   EXPECT_EQ(ui::ET_MOUSE_PRESSED, recorder_second.events()[0]);
   EXPECT_EQ(event_location.ToString(),
             recorder_second.mouse_locations()[0].ToString());
 }
@@ skipping 55 matching lines @@
   EXPECT_EQ(gfx::Point(kX - kWindowOffset, kY - kWindowOffset).ToString(),
             recorder.gesture_locations()[0].ToString());
   root_window()->RemovePreTargetHandler(&recorder);
 }
 
 // Tests that a scroll-generating touch-event is marked as such.
 TEST_F(WindowEventDispatcherTest, TouchMovesMarkedWhenCausingScroll) {
   EventFilterRecorder recorder;
   root_window()->AddPreTargetHandler(&recorder);
 
-  const gfx::Point location(20, 20);
+  const gfx::PointF location(20.f, 20.f);
   ui::TouchEvent press(
       ui::ET_TOUCH_PRESSED, location, 0, ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&press);
   EXPECT_FALSE(recorder.LastTouchMayCauseScrolling());
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_TOUCH_PRESSED));
   recorder.Reset();
 
   ui::TouchEvent move(ui::ET_TOUCH_MOVED,
-                      location + gfx::Vector2d(100, 100),
-                      0,
+                      location + gfx::Vector2dF(100.f, 100.f), 0,
                       ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&move);
   EXPECT_TRUE(recorder.LastTouchMayCauseScrolling());
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_TOUCH_MOVED));
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_GESTURE_SCROLL_BEGIN));
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_GESTURE_SCROLL_UPDATE));
   recorder.Reset();
 
   ui::TouchEvent move2(ui::ET_TOUCH_MOVED,
-                       location + gfx::Vector2d(200, 200),
-                       0,
+                       location + gfx::Vector2dF(200.f, 200.f), 0,
                        ui::EventTimeForNow());
   DispatchEventUsingWindowDispatcher(&move2);
   EXPECT_TRUE(recorder.LastTouchMayCauseScrolling());
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_TOUCH_MOVED));
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_GESTURE_SCROLL_UPDATE));
   recorder.Reset();
 
   // Delay the release to avoid fling generation.
   ui::TouchEvent release(
-      ui::ET_TOUCH_RELEASED,
-      location + gfx::Vector2dF(200, 200),
-      0,
+      ui::ET_TOUCH_RELEASED, location + gfx::Vector2dF(200.f, 200.f), 0,
       ui::EventTimeForNow() + base::TimeDelta::FromSeconds(1));
   DispatchEventUsingWindowDispatcher(&release);
   EXPECT_FALSE(recorder.LastTouchMayCauseScrolling());
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_TOUCH_RELEASED));
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_GESTURE_SCROLL_END));
 
   root_window()->RemovePreTargetHandler(&recorder);
 }
 
 // OnCursorMovedToRootLocation() is sometimes called instead of
@@ skipping 21 matching lines @@
   // 2x. A ET_MOUSE_EXITED event should have been sent to |w|.
   test_screen()->SetDeviceScaleFactor(2.f);
   dispatcher->OnCursorMovedToRootLocation(gfx::Point(11, 11));
   RunAllPendingInMessageLoop();
   EXPECT_TRUE(recorder.HasReceivedEvent(ui::ET_MOUSE_EXITED));
 
   w->RemovePreTargetHandler(&recorder);
 }
 
 }  // namespace aura