Add Vector2d classes that represent offsets, instead of using Point.

ui/aura/test/event_generator.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 "ui/aura/test/event_generator.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "ui/aura/root_window.h"
 #include "ui/base/events/event.h"
+#include "ui/gfx/vector2d_conversions.h"
 
 #if defined(USE_X11)
 #include <X11/Xlib.h>
 #include "ui/base/x/x11_util.h"
 #endif
 
 #if defined(OS_WIN)
 #include "ui/base/keycodes/keyboard_code_conversion.h"
 #endif
 
@@ skipping 99 matching lines @@
                            current_location_, flags_);
     Dispatch(mouseev);
     flags_ ^= ui::EF_RIGHT_MOUSE_BUTTON;
   }
 }
 
 void EventGenerator::MoveMouseTo(const gfx::Point& point, int count) {
   DCHECK_GT(count, 0);
   const ui::EventType event_type = (flags_ & ui::EF_LEFT_MOUSE_BUTTON) ?
       ui::ET_MOUSE_DRAGGED : ui::ET_MOUSE_MOVED;
-  const gfx::Point diff = point.Subtract(current_location_);
   for (int i = 1; i <= count; i++) {
-    const gfx::Point move_point = current_location_.Add(
-        gfx::Point(diff.x() / count * i, diff.y() / count * i));
+    gfx::Vector2dF diff = point - current_location_;

[Peter Kasting, 2012/10/30 21:41:23]

This is wrong if one of the mouse dispatch event processing calls ends up
changing current_location_.  I don't know if that can happen but I'd rather not
risk it.  How about:

  gfx::Vector2d diff(point - current_location_);
  for (float i = 1; i <= count; ++i) {
    gfx::Vector2dF step(diff);
    step.Scale(i / count);
    gfx::Point event_point(current_location_ + gfx::ToRoundedVector2d(step));
    Dispatch(ui::MouseEvent(event_type, event_point, event_point, flags));
  }

[danakj, 2012/10/30 21:49:10]

ya, ok!

+    diff.Scale(static_cast<float>(i) / count);
+    gfx::Point move_point = current_location_ + gfx::ToFlooredVector2d(diff);
     ui::MouseEvent mouseev(event_type, move_point, move_point, flags_);
     Dispatch(mouseev);
   }
   current_location_ = point;
 }
 
 void EventGenerator::MoveMouseRelativeTo(const Window* window,
                                          const gfx::Point& point) {
   gfx::Point root_point(point);
   Window::ConvertPointToTarget(window, root_window_, &root_point);
@@ skipping 192 matching lines @@
   TestKeyEvent keyev(native_event.get(), flags, false);
 #else
   ui::KeyEvent keyev(type, key_code, flags, false);
 #endif  // USE_X11
 #endif  // OS_WIN
   Dispatch(keyev);
 }
 
 }  // namespace test
 }  // namespace aura