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

ui/events/event_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 "base/memory/scoped_ptr.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/events/event.h"
 #include "ui/events/event_utils.h"
 #include "ui/events/keycodes/dom/dom_code.h"
 #include "ui/events/keycodes/dom/keycode_converter.h"
@@ skipping 624 matching lines @@
   EXPECT_EQ(EventPointerType::POINTER_TYPE_TOUCH,
             touch_event_copy.pointer_details().pointer_type());
   EXPECT_EQ(10.0f, touch_event_copy.pointer_details().radius_x());
   EXPECT_EQ(5.0f, touch_event_copy.pointer_details().radius_y());
   EXPECT_EQ(15.0f, touch_event_copy.pointer_details().force());
   EXPECT_EQ(0.0f, touch_event_copy.pointer_details().tilt_x());
   EXPECT_EQ(0.0f, touch_event_copy.pointer_details().tilt_y());
 }
 
 TEST(EventTest, PointerEventDetailsMouse) {
-  ui::MouseEvent mouse_event(ET_MOUSE_PRESSED, gfx::PointF(0, 0),
-                             gfx::PointF(0, 0), ui::EventTimeForNow(), 0, 0);
+  ui::MouseEvent mouse_event(ET_MOUSE_PRESSED, gfx::Point(0, 0),

[sky, 2015/10/30 22:20:44]

0,0 is synonymous with gfx::Point()

[danakj, 2015/10/30 22:39:45]

Ya, it is. vmpstr recently convinced me that writing Point(0, 0) is actually
nicer to the reader than Point() when you care about the value of the thing, so
I stopped removing 0,0s where they exist.

+                             gfx::Point(0, 0), ui::EventTimeForNow(), 0, 0);
 
   EXPECT_EQ(EventPointerType::POINTER_TYPE_MOUSE,
             mouse_event.pointer_details().pointer_type());
   EXPECT_EQ(0.0f, mouse_event.pointer_details().radius_x());
   EXPECT_EQ(0.0f, mouse_event.pointer_details().radius_y());
   EXPECT_EQ(0.0f, mouse_event.pointer_details().force());
   EXPECT_EQ(0.0f, mouse_event.pointer_details().tilt_x());
   EXPECT_EQ(0.0f, mouse_event.pointer_details().tilt_y());
 
   ui::MouseEvent mouse_event_copy(mouse_event);
   EXPECT_EQ(EventPointerType::POINTER_TYPE_MOUSE,
             mouse_event_copy.pointer_details().pointer_type());
   EXPECT_EQ(0.0f, mouse_event_copy.pointer_details().radius_x());
   EXPECT_EQ(0.0f, mouse_event_copy.pointer_details().radius_y());
   EXPECT_EQ(0.0f, mouse_event_copy.pointer_details().force());
   EXPECT_EQ(0.0f, mouse_event_copy.pointer_details().tilt_x());
   EXPECT_EQ(0.0f, mouse_event_copy.pointer_details().tilt_y());
 }
 
 TEST(EventTest, PointerEventDetailsStylus) {
-  ui::MouseEvent stylus_event(ET_MOUSE_PRESSED, gfx::PointF(0, 0),
-                              gfx::PointF(0, 0), ui::EventTimeForNow(), 0, 0);
+  ui::MouseEvent stylus_event(ET_MOUSE_PRESSED, gfx::Point(0, 0),
+                              gfx::Point(0, 0), ui::EventTimeForNow(), 0, 0);
   ui::PointerDetails pointer_details(EventPointerType::POINTER_TYPE_PEN,
       /* radius_x */ 0.0f,
       /* radius_y */ 0.0f,
       /* force */ 21.0f,
       /* tilt_x */ 45.0f,
       /* tilt_y */ -45.0f);
 
   stylus_event.set_pointer_details(pointer_details);
   EXPECT_EQ(EventPointerType::POINTER_TYPE_PEN,
             stylus_event.pointer_details().pointer_type());
   EXPECT_EQ(21.0f, stylus_event.pointer_details().force());
   EXPECT_EQ(45.0f, stylus_event.pointer_details().tilt_x());
   EXPECT_EQ(-45.0f, stylus_event.pointer_details().tilt_y());
   EXPECT_EQ(0.0f, stylus_event.pointer_details().radius_x());
   EXPECT_EQ(0.0f, stylus_event.pointer_details().radius_y());
 
   ui::MouseEvent stylus_event_copy(stylus_event);
   EXPECT_EQ(EventPointerType::POINTER_TYPE_PEN,
             stylus_event_copy.pointer_details().pointer_type());
   EXPECT_EQ(21.0f, stylus_event_copy.pointer_details().force());
   EXPECT_EQ(45.0f, stylus_event_copy.pointer_details().tilt_x());
   EXPECT_EQ(-45.0f, stylus_event_copy.pointer_details().tilt_y());
   EXPECT_EQ(0.0f, stylus_event_copy.pointer_details().radius_x());
   EXPECT_EQ(0.0f, stylus_event_copy.pointer_details().radius_y());
 }
 
 }  // namespace ui