Use local coordinates in TouchEditable interface

ui/base/touch/touch_editing_controller.h

 // Copyright (c) 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.
 
 #ifndef UI_BASE_TOUCH_TOUCH_EDITING_CONTROLLER_H_
 #define UI_BASE_TOUCH_TOUCH_EDITING_CONTROLLER_H_
 
 #include "ui/base/models/simple_menu_model.h"
 #include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
 
 namespace ui {
 
 // An interface implemented by widget that has text that can be selected/edited
 // using touch.
 class UI_BASE_EXPORT TouchEditable : public ui::SimpleMenuModel::Delegate {
  public:
+  // TODO(mohsen): Consider switching from local coordinates to screen
+  // coordinates in this interface and see if it will simplify things.
+
   // Select everything between start and end (points are in view's local
   // coordinate system). |start| is the logical start and |end| is the logical
   // end of selection. Visually, |start| may lie after |end|.
   virtual void SelectRect(const gfx::Point& start, const gfx::Point& end) = 0;
 
   // Move the caret to |point|. |point| is in local coordinates.
   virtual void MoveCaretTo(const gfx::Point& point) = 0;
 
   // Gets the end points of the current selection. The end points p1 and p2 must
-  // be the cursor rect for the start and end of selection:
+  // be the cursor rect for the start and end of selection (in local
+  // coordinates):
   // ____________________________________
   // | textfield with |selected text|   |
   // ------------------------------------
   //                  ^p1           ^p2
   //
   // p1 should be the logical start and p2 the logical end of selection. Hence,
   // visually, p1 could be to the right of p2 in the figure above.
   virtual void GetSelectionEndPoints(gfx::Rect* p1, gfx::Rect* p2) = 0;
 
-  // Gets the bounds of the client view in parent's coordinates.
+  // Gets the bounds of the client view in its local coordinates.
   virtual gfx::Rect GetBounds() = 0;
 
   // Gets the NativeView hosting the client.
   virtual gfx::NativeView GetNativeView() const = 0;
 
   // Converts a point to/from screen coordinates from/to client view.
   virtual void ConvertPointToScreen(gfx::Point* point) = 0;
   virtual void ConvertPointFromScreen(gfx::Point* point) = 0;
 
   // Returns true if the editable draws its own handles (hence, the
@@ skipping 30 matching lines @@
 
   virtual TouchSelectionController* create(TouchEditable* client_view) = 0;
 
  protected:
   virtual ~TouchSelectionControllerFactory() {}
 };
 
 }  // namespace views
 
 #endif  // UI_BASE_TOUCH_TOUCH_EDITING_CONTROLLER_H_