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

ui/views/controls/table/table_view_win.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/views/controls/table/table_view_win.h"
 
 #include <commctrl.h>
 #include <windowsx.h>
 
 #include <algorithm>
@@ skipping 499 matching lines @@
                                          LPARAM l_param) {
   TableView* table_view = reinterpret_cast<TableViewWrapper*>(
       GetWindowLongPtr(window, GWLP_USERDATA))->table_view;
 
   // Is the mouse down on the table?
   static bool in_mouse_down = false;
   // Should we select on mouse up?
   static bool select_on_mouse_up = false;
 
   // If the mouse is down, this is the location of the mouse down message.
-  static int mouse_down_x, mouse_down_y;
+  CR_DEFINE_STATIC_LOCAL(gfx::Point, mouse_down_pos, ());
 
   switch (message) {
     case WM_CONTEXTMENU: {
       // This addresses two problems seen with context menus in right to left
       // locales:
       // 1. The mouse coordinates in the l_param were occasionally wrong in
       //    weird ways. This is most often seen when right clicking on the
       //    list-view twice in a row.
       // 2. Right clicking on the icon would show the scrollbar menu.
       //
@@ skipping 109 matching lines @@
       if ((w_param | (MK_LBUTTON | MK_CONTROL | MK_SHIFT)) ==
           (MK_LBUTTON | MK_CONTROL | MK_SHIFT)) {
         if (in_mouse_down)
           return 0;
 
         int view_index = GetViewIndexFromPoint(window, gfx::Point(l_param));
         if (view_index != -1) {
           table_view->ignore_listview_change_ = true;
           in_mouse_down = true;
           select_on_mouse_up = false;
-          mouse_down_x = GET_X_LPARAM(l_param);
-          mouse_down_y = GET_Y_LPARAM(l_param);
+          mouse_down_pos.set_x(GET_X_LPARAM(l_param));
+          mouse_down_pos.set_y(GET_Y_LPARAM(l_param));
           int model_index = table_view->ViewToModel(view_index);
           bool select = true;
           if (w_param & MK_CONTROL) {
             select = false;
             if (!table_view->IsItemSelected(model_index)) {
               if (table_view->single_selection_) {
                 // Single selection mode and the row isn't selected, select
                 // only it.
                 table_view->Select(model_index);
               } else {
@@ skipping 40 matching lines @@
           SetCapture(window);
           return 0;
         }
         // else case, continue on to default handler
       }
       break;
     }
 
     case WM_MOUSEMOVE: {
       if (in_mouse_down) {
-        int x = GET_X_LPARAM(l_param);
-        int y = GET_Y_LPARAM(l_param);
-        if (View::ExceededDragThreshold(x - mouse_down_x, y - mouse_down_y)) {
+        gfx::Point mouse_pos(GET_X_LPARAM(l_param), GET_Y_LPARAM(l_param));
+        if (View::ExceededDragThreshold(mouse_pos - mouse_down_pos)) {
           // We're about to start drag and drop, which results in no mouse up.
           // Release capture and reset state.
           ReleaseCapture();
           in_mouse_down = false;
 
           NMLISTVIEW details;
           memset(&details, 0, sizeof(details));
           details.hdr.code = LVN_BEGINDRAG;
           SendMessage(::GetParent(window), WM_NOTIFY, 0,
                       reinterpret_cast<LPARAM>(&details));
@@ skipping 916 matching lines @@
 }
 
 void TableSelectionIterator::UpdateModelIndexFromViewIndex() {
   if (view_index_ == -1)
     model_index_ = -1;
   else
     model_index_ = table_view_->ViewToModel(view_index_);
 }
 
 }  // namespace views