Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(736)

Unified Diff: chrome/browser/ui/autofill/autofill_popup_controller.h

Issue 44543002: Enable touch for autofill popup view (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Back to "selection" nomenclature Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/autofill/autofill_popup_controller.h
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller.h b/chrome/browser/ui/autofill/autofill_popup_controller.h
index 4b4f7c6d8f540ce1d031f13100dc8f35b77504df..0d07125ed06cb219a6c85c8ff0811ce2c8851b9d 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller.h
+++ b/chrome/browser/ui/autofill/autofill_popup_controller.h
@@ -36,14 +36,14 @@ class AutofillPopupController {
// 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 selected (e.g., mouse hovered).
Ilya Sherman 2013/11/05 18:39:45 nit: "is" -> "was"; or better yet, something like
mohsen 2013/11/05 19:52:36 Done.
+ virtual void LineSelectedAtPoint(int x, int y) = 0;
- // The user clicked the mouse within the popup.
- virtual void MouseClicked(int x, int y) = 0;
+ // The line at (x, y) is accepted (e.g., mouse clicked).
+ virtual void LineAcceptedAtPoint(int x, int y) = 0;
- // The user has moved the mouse outside of the popup.
- virtual void MouseExitedPopup() = 0;
+ // The selection is cleared (e.g., mouse exited popup).
+ virtual void SelectionCleared() = 0;
// Whether |event| should be reposted to the native window management.
virtual bool ShouldRepostEvent(const ui::MouseEvent& event) = 0;

Powered by Google App Engine
This is Rietveld 408576698