Explicitly convert Point to PointF for event code

ash/drag_drop/drag_drop_controller.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 "ash/drag_drop/drag_drop_controller.h"
 
 #include "ash/drag_drop/drag_drop_tracker.h"
 #include "ash/drag_drop/drag_image_view.h"
 #include "ash/shell.h"
 #include "base/bind.h"
@@ skipping 33 matching lines @@
 static const int kTouchDragImageVerticalOffset = -25;
 
 // Adjusts the drag image bounds such that the new bounds are scaled by |scale|
 // and translated by the |drag_image_offset| and and additional
 // |vertical_offset|.
 gfx::Rect AdjustDragImageBoundsForScaleAndOffset(
     const gfx::Rect& drag_image_bounds,
     int vertical_offset,
     float scale,
     gfx::Vector2d* drag_image_offset) {
-  gfx::PointF final_origin = drag_image_bounds.origin();
-  gfx::SizeF final_size = gfx::SizeF(drag_image_bounds.size());
+  gfx::PointF final_origin(drag_image_bounds.origin());

[sky, 2015/10/23 21:13:13]

You don't need the floating point until the RectF constructor below. So, keep
this a Point until then.

[danakj, 2015/10/23 21:46:10]

Done.

+  gfx::SizeF final_size(drag_image_bounds.size());
   final_size.Scale(scale);
   drag_image_offset->set_x(drag_image_offset->x() * scale);
   drag_image_offset->set_y(drag_image_offset->y() * scale);
   float total_x_offset = drag_image_offset->x();

[danakj, 2015/10/23 21:46:10]

made these ints too

   float total_y_offset = drag_image_offset->y() - vertical_offset;
   final_origin.Offset(-total_x_offset, -total_y_offset);
   return gfx::ToEnclosingRect(gfx::RectF(final_origin, final_size));
 }
 
 void DispatchGestureEndToWindow(aura::Window* window) {
   if (window && window->delegate()) {
     ui::GestureEvent gesture_end(0,
                                  0,
                                  0,
@@ skipping 485 matching lines @@
   if (drag_window_)
     drag_window_->RemoveObserver(this);
   drag_window_ = NULL;
   drag_data_ = NULL;
   // Cleanup can be called again while deleting DragDropTracker, so delete
   // the pointer with a local variable to avoid double free.
   scoped_ptr<ash::DragDropTracker> holder = drag_drop_tracker_.Pass();
 }
 
 }  // namespace ash