Enable touch for autofill popup view

chrome/browser/ui/autofill/autofill_popup_controller.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_CONTROLLER_H_
 #define CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_CONTROLLER_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 18 matching lines @@
  public:
   // Called when the popup should get hidden.
   virtual void Hide() = 0;
 
   // Called whent the popup view was destroyed.
   virtual void ViewDestroyed() = 0;
 
   // Recalculates the height and width of the popup and triggers a redraw.
   virtual void UpdateBoundsAndRedrawPopup() = 0;
 
-  // The user has moved the mouse within the popup.
-  virtual void MouseHovered(int x, int y) = 0;
+  // The line at (x, y) is preselected (e.g., mouse hovered).
+  virtual void PointPreselected(int x, int y) = 0;

[Ilya Sherman, 2013/11/01 06:45:50]

It doesn't make sense, at least in terminology that I am aware of, to "select"
or "preselect" a point.  Please find better names for these methods, or possibly
just keep the Mouse* methods, and add touch-specific ones.

 
-  // The user clicked the mouse within the popup.
-  virtual void MouseClicked(int x, int y) = 0;
+  // The line at (x, y) is selected (e.g., mouse clicked).
+  virtual void PointSelected(int x, int y) = 0;
 
-  // The user has moved the mouse outside of the popup.
-  virtual void MouseExitedPopup() = 0;
+  // The preselection is cleared (e.g., mouse exited popup).
+  virtual void PreselectionCleared() = 0;
 
   // Whether |event| should be reposted to the native window management.
   virtual bool ShouldRepostEvent(const ui::MouseEvent& event) = 0;
 
   // Accepts the suggestion at |index|.
   virtual void AcceptSuggestion(size_t index) = 0;
 
   // Gets the resource value for the given resource, returning -1 if the
   // resource isn't recognized.
   virtual int GetIconResourceID(const string16& resource_name) = 0;
@@ skipping 39 matching lines @@
   // Identifier for the row.
   virtual const std::vector<int>& identifiers() const = 0;
 
 #if !defined(OS_ANDROID)
   // The same font can vary based on the type of data it is showing,
   // so we need to know the row.
   virtual const gfx::Font& GetNameFontForRow(size_t index) const = 0;
   virtual const gfx::Font& subtext_font() const = 0;
 #endif
 
-  // Returns the index of the selected line. A line is "selected" when it is
-  // hovered or has keyboard focus.
-  virtual int selected_line() const = 0;
+  // Returns the index of the preselected line. A line is "preselected" when it
+  // is hovered or has keyboard focus.
+  virtual int preselected_line() const = 0;
 
   // Whether the view should be hidden on outside mouse presses.
   virtual bool hide_on_outside_click() const = 0;
 
  protected:
   virtual ~AutofillPopupController() {}
 };
 
 }  // namespace autofill
 
 #endif  // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_CONTROLLER_H_