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

ui/aura/window_targeter_unittest.cc

 // Copyright (c) 2013 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_targeter.h"
 
 #include "ui/aura/scoped_window_targeter.h"
 #include "ui/aura/test/aura_test_base.h"
 #include "ui/aura/test/test_window_delegate.h"
 #include "ui/aura/window.h"
@@ skipping 49 matching lines @@
 
   window->SetBounds(gfx::Rect(0, 0, 100, 100));
   one->SetBounds(gfx::Rect(0, 0, 500, 100));
   two->SetBounds(gfx::Rect(501, 0, 500, 1000));
 
   root_window()->Show();
 
   ui::test::TestEventHandler handler;
   one->AddPreTargetHandler(&handler);
 
-  ui::MouseEvent press(ui::ET_MOUSE_PRESSED, gfx::Point(20, 20),
-                       gfx::Point(20, 20), ui::EventTimeForNow(), ui::EF_NONE,
-                       ui::EF_NONE);
+  ui::MouseEvent press(ui::ET_MOUSE_PRESSED, gfx::PointF(20.f, 20.f),
+                       gfx::PointF(20.f, 20.f), ui::EventTimeForNow(),
+                       ui::EF_NONE, ui::EF_NONE);
   DispatchEventUsingWindowDispatcher(&press);
   EXPECT_EQ(1, handler.num_mouse_events());
 
   handler.Reset();
   DispatchEventUsingWindowDispatcher(&press);
   EXPECT_EQ(1, handler.num_mouse_events());
 
   one->RemovePreTargetHandler(&handler);
 }
 
 TEST_F(WindowTargeterTest, ScopedWindowTargeter) {
   test::TestWindowDelegate delegate;
   scoped_ptr<Window> window(CreateNormalWindow(1, root_window(), &delegate));
   Window* child = CreateNormalWindow(2, window.get(), &delegate);
 
   window->SetBounds(gfx::Rect(30, 30, 100, 100));
   child->SetBounds(gfx::Rect(20, 20, 50, 50));
   root_window()->Show();
 
   ui::EventTarget* root = root_window();
   ui::EventTargeter* targeter = root->GetEventTargeter();
 
-  gfx::Point event_location(60, 60);
+  gfx::PointF event_location(60.f, 60.f);
   {
     ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, event_location, event_location,
                          ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
     EXPECT_EQ(child, targeter->FindTargetForEvent(root, &mouse));
   }
 
   // Install a targeter on |window| so that the events never reach the child.
   scoped_ptr<ScopedWindowTargeter> scoped_targeter(
       new ScopedWindowTargeter(window.get(), scoped_ptr<ui::EventTargeter>(
           new StaticWindowTargeter(window.get()))));
@@ skipping 29 matching lines @@
   root_window()->Show();
 
   test::TestWindowDelegate delegate;
   scoped_ptr<Window> window(CreateNormalWindow(2, root_window(), &delegate));
 
   const gfx::Rect window_bounds(100, 20, 400, 80);
   window->SetBounds(window_bounds);
 
   ui::EventTarget* root_target = root_window();
   ui::EventTargeter* targeter = root_target->GetEventTargeter();
-  gfx::Point event_location(490, 50);
+  gfx::PointF event_location(490.f, 50.f);
   {
     ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, event_location, event_location,
                          ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
     EXPECT_EQ(window.get(), targeter->FindTargetForEvent(root_target, &mouse));
   }
 
   // Scale |window| by 50%. This should move it away from underneath
   // |event_location|, so an event in that location will not be targeted to it.
   gfx::Transform transform;
   transform.Scale(0.5, 0.5);
@@ skipping 53 matching lines @@
   Window* parent_r = root_window()->children()[0];
   Window* child_r = parent_r->children()[0];
   Window* grandchild_r = child_r->children()[0];
 
   ui::EventTarget* root_target = root_window();
   ui::EventTargeter* targeter = root_target->GetEventTargeter();
 
   // Dispatch a mouse event that falls on the parent, but not on the child. When
   // the default event-targeter used, the event will still reach |grandchild|,
   // because the default targeter does not look at the bounds.
-  ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, gfx::Point(1, 1), gfx::Point(1, 1),
-                       ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
+  ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, gfx::PointF(1.f, 1.f),
+                       gfx::PointF(1.f, 1.f), ui::EventTimeForNow(),
+                       ui::EF_NONE, ui::EF_NONE);
   EXPECT_EQ(parent_r, targeter->FindTargetForEvent(root_target, &mouse));
 
   // Install a targeter on the |child| that looks at the window id as well
   // as the bounds and makes sure the event reaches the target only if the id of
   // the window is equal to 2 (incorrect). This causes the event to get handled
   // by |parent|.
-  ui::MouseEvent mouse2(ui::ET_MOUSE_MOVED, gfx::Point(8, 8), gfx::Point(8, 8),
-                        ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
+  ui::MouseEvent mouse2(ui::ET_MOUSE_MOVED, gfx::PointF(8.f, 8.f),
+                        gfx::PointF(8.f, 8.f), ui::EventTimeForNow(),
+                        ui::EF_NONE, ui::EF_NONE);
   scoped_ptr<ui::EventTargeter> original_targeter = child_r->SetEventTargeter(
       scoped_ptr<ui::EventTargeter>(new IdCheckingEventTargeter(2)));
   EXPECT_EQ(parent_r, targeter->FindTargetForEvent(root_target, &mouse2));
 
   // Now install a targeter on the |child| that looks at the window id as well
   // as the bounds and makes sure the event reaches the target only if the id of
   // the window is equal to 1 (correct).
-  ui::MouseEvent mouse3(ui::ET_MOUSE_MOVED, gfx::Point(8, 8), gfx::Point(8, 8),
-                        ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
+  ui::MouseEvent mouse3(ui::ET_MOUSE_MOVED, gfx::PointF(8.f, 8.f),
+                        gfx::PointF(8.f, 8.f), ui::EventTimeForNow(),
+                        ui::EF_NONE, ui::EF_NONE);
   child_r->SetEventTargeter(
       scoped_ptr<ui::EventTargeter>(new IdCheckingEventTargeter(1)));
   EXPECT_EQ(child_r, targeter->FindTargetForEvent(root_target, &mouse3));
 
   // restore original WindowTargeter for |child|.
   child_r->SetEventTargeter(original_targeter.Pass());
 
   // Target |grandchild| location.
-  ui::MouseEvent second(ui::ET_MOUSE_MOVED, gfx::Point(12, 12),
-                        gfx::Point(12, 12), ui::EventTimeForNow(), ui::EF_NONE,
-                        ui::EF_NONE);
+  ui::MouseEvent second(ui::ET_MOUSE_MOVED, gfx::PointF(12.f, 12.f),
+                        gfx::PointF(12.f, 12.f), ui::EventTimeForNow(),
+                        ui::EF_NONE, ui::EF_NONE);
   EXPECT_EQ(grandchild_r, targeter->FindTargetForEvent(root_target, &second));
 
   // Target |child| location.
-  ui::MouseEvent third(ui::ET_MOUSE_MOVED, gfx::Point(8, 8), gfx::Point(8, 8),
-                       ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
+  ui::MouseEvent third(ui::ET_MOUSE_MOVED, gfx::PointF(8.f, 8.f),
+                       gfx::PointF(8.f, 8.f), ui::EventTimeForNow(),
+                       ui::EF_NONE, ui::EF_NONE);
   EXPECT_EQ(child_r, targeter->FindTargetForEvent(root_target, &third));
 }
 
 class IgnoreWindowTargeter : public WindowTargeter {
  public:
   IgnoreWindowTargeter() {}
   ~IgnoreWindowTargeter() override {}
 
  private:
   // WindowTargeter:
   bool SubtreeShouldBeExploredForEvent(Window* window,
                                        const ui::LocatedEvent& event) override {
     return false;
   }
 };
 
 // Verifies that an EventTargeter installed on an EventTarget can dictate
 // whether the target itself can process an event.
 TEST_F(WindowTargeterTest, TargeterChecksOwningEventTarget) {
   test::TestWindowDelegate delegate;
   scoped_ptr<Window> child(CreateNormalWindow(1, root_window(), &delegate));
 
   ui::EventTarget* root_target = root_window();
   ui::EventTargeter* targeter = root_target->GetEventTargeter();
 
-  ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, gfx::Point(10, 10),
-                       gfx::Point(10, 10), ui::EventTimeForNow(), ui::EF_NONE,
-                       ui::EF_NONE);
+  ui::MouseEvent mouse(ui::ET_MOUSE_MOVED, gfx::PointF(10.f, 10.f),
+                       gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
+                       ui::EF_NONE, ui::EF_NONE);
   EXPECT_EQ(child.get(), targeter->FindTargetForEvent(root_target, &mouse));
 
   // Install an event targeter on |child| which always prevents the target from
   // receiving event.
   child->SetEventTargeter(
       scoped_ptr<ui::EventTargeter>(new IgnoreWindowTargeter()));
 
-  ui::MouseEvent mouse2(ui::ET_MOUSE_MOVED, gfx::Point(10, 10),
-                        gfx::Point(10, 10), ui::EventTimeForNow(), ui::EF_NONE,
-                        ui::EF_NONE);
+  ui::MouseEvent mouse2(ui::ET_MOUSE_MOVED, gfx::PointF(10.f, 10.f),
+                        gfx::PointF(10.f, 10.f), ui::EventTimeForNow(),
+                        ui::EF_NONE, ui::EF_NONE);
   EXPECT_EQ(root_window(), targeter->FindTargetForEvent(root_target, &mouse2));
 }
 
 }  // namespace aura