Handle floating point coordinates from ozone to exosphere

ui/views/view.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.
 
 #define _USE_MATH_DEFINES // For VC++ to get M_PI. This has to be first.
 
 #include "ui/views/view.h"
 
 #include <algorithm>
 #include <cmath>
@@ skipping 690 matching lines @@
   GetWidget()->GetRootView()->GetViewsInGroup(group, &views);
   return views.empty() ? NULL : views[0];
 }
 
 // Coordinate conversion -------------------------------------------------------
 
 // static
 void View::ConvertPointToTarget(const View* source,
                                 const View* target,
                                 gfx::Point* point) {
+  gfx::PointF point_f(*point);
+  ConvertPointToTargetF(source, target, &point_f);
+  *point = gfx::ToFlooredPoint(point_f);
+}
+
+// static
+void View::ConvertPointToTargetF(const View* source,
+                                 const View* target,
+                                 gfx::PointF* point) {
   DCHECK(source);
   DCHECK(target);
   if (source == target)
     return;
 
   const View* root = GetHierarchyRoot(target);
   CHECK_EQ(GetHierarchyRoot(source), root);
 
   if (source != root)
-    source->ConvertPointForAncestor(root, point);
+    source->ConvertPointForAncestorF(root, point);
 
   if (target != root)
-    target->ConvertPointFromAncestor(root, point);
+    target->ConvertPointFromAncestorF(root, point);
 }
 
 // static
 void View::ConvertRectToTarget(const View* source,
                                const View* target,
                                gfx::RectF* rect) {
   DCHECK(source);
   DCHECK(target);
   if (source == target)
     return;
@@ skipping 1405 matching lines @@
     p = p->parent_;
   }
 
   return p == ancestor;
 }
 
 // Coordinate conversion -------------------------------------------------------
 
 bool View::ConvertPointForAncestor(const View* ancestor,
                                    gfx::Point* point) const {
+  gfx::PointF point_f(*point);
+  bool result = ConvertPointForAncestorF(ancestor, &point_f);
+  *point = gfx::ToFlooredPoint(point_f);
+  return result;
+}
+
+bool View::ConvertPointForAncestorF(const View* ancestor,
+                                    gfx::PointF* point) const {
   gfx::Transform trans;
   // TODO(sad): Have some way of caching the transformation results.
   bool result = GetTransformRelativeTo(ancestor, &trans);
-  auto p = gfx::Point3F(gfx::PointF(*point));
+  auto p = gfx::Point3F(*point);
   trans.TransformPoint(&p);
-  *point = gfx::ToFlooredPoint(p.AsPointF());
+  *point = p.AsPointF();
   return result;
 }
 
 bool View::ConvertPointFromAncestor(const View* ancestor,
                                     gfx::Point* point) const {
-  gfx::Transform trans;
-  bool result = GetTransformRelativeTo(ancestor, &trans);
-  auto p = gfx::Point3F(gfx::PointF(*point));
-  trans.TransformPointReverse(&p);
-  *point = gfx::ToFlooredPoint(p.AsPointF());
+  gfx::PointF point_f(*point);
+  bool result = ConvertPointFromAncestorF(ancestor, &point_f);
+  *point = gfx::ToFlooredPoint(point_f);
   return result;
 }
 
+bool View::ConvertPointFromAncestorF(const View* ancestor,
+                                     gfx::PointF* point) const {
+  gfx::Transform trans;
+  bool result = GetTransformRelativeTo(ancestor, &trans);
+  auto p = gfx::Point3F(*point);
+  trans.TransformPointReverse(&p);
+  *point = p.AsPointF();
+  return result;
+}
+
 bool View::ConvertRectForAncestor(const View* ancestor,
                                   gfx::RectF* rect) const {
   gfx::Transform trans;
   // TODO(sad): Have some way of caching the transformation results.
   bool result = GetTransformRelativeTo(ancestor, &trans);
   trans.TransformRect(rect);
   return result;
 }
 
 bool View::ConvertRectFromAncestor(const View* ancestor,
@@ skipping 368 matching lines @@
   // Message the RootView to do the drag and drop. That way if we're removed
   // the RootView can detect it and avoid calling us back.
   gfx::Point widget_location(event.location());
   ConvertPointToWidget(this, &widget_location);
   widget->RunShellDrag(this, data, widget_location, drag_operations, source);
   // WARNING: we may have been deleted.
   return true;
 }
 
 }  // namespace views