views: Update event-related API to use PointF/RectF instead of Point/Rect.

ui/views/rect_based_targeting_utils.cc

 // Copyright 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/views/rect_based_targeting_utils.h"
 
 #include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
 
 namespace views {
 
-bool UsePointBasedTargeting(const gfx::Rect& rect) {
-  return rect.width() == 1 && rect.height() == 1;
+bool UsePointBasedTargeting(const gfx::RectF& rect) {
+  return rect.width() == 1.f && rect.height() == 1.f;

[tdresser, 2014/05/02 13:19:06]

Is comparing equality of floats here acceptable, or should we use a proper
floating point comparison?

 }
 
-float PercentCoveredBy(const gfx::Rect& rect_1, const gfx::Rect& rect_2) {
-  gfx::Rect intersection(rect_1);
+float PercentCoveredBy(const gfx::RectF& rect_1, const gfx::RectF& rect_2) {
+  gfx::RectF intersection(rect_1);
   intersection.Intersect(rect_2);
   int intersect_area = intersection.size().GetArea();
   int rect_1_area = rect_1.size().GetArea();
   return rect_1_area ?
       static_cast<float>(intersect_area) / static_cast<float>(rect_1_area) : 0;
 }
 
-int DistanceSquaredFromCenterToPoint(const gfx::Point& point,
-                                     const gfx::Rect& rect) {
-  gfx::Point center_point = rect.CenterPoint();
-  int dx = center_point.x() - point.x();
-  int dy = center_point.y() - point.y();
+float DistanceSquaredFromCenterToPoint(const gfx::PointF& point,
+                                       const gfx::RectF& rect) {
+  gfx::PointF center_point = rect.CenterPoint();
+  float dx = center_point.x() - point.x();
+  float dy = center_point.y() - point.y();
   return (dx * dx) + (dy * dy);
 }
 
 }  // namespace views