Enable touch for autofill popup view

chrome/browser/ui/views/autofill/autofill_popup_view_views.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 "chrome/browser/ui/views/autofill/autofill_popup_view_views.h"
 
 #include "chrome/browser/ui/autofill/autofill_popup_controller.h"
 #include "grit/ui_resources.h"
 #include "third_party/WebKit/public/web/WebAutofillClient.h"
 #include "ui/base/resource/resource_bundle.h"
@@ skipping 144 matching lines @@
   // receive an OnMouseReleased event just after showing. This breaks the mouse
   // capture, so restart capturing here.
   if (controller_->hide_on_outside_click() && GetWidget())
     GetWidget()->SetCapture(this);
 
   // We only care about the left click.
   if (event.IsOnlyLeftMouseButton() && HitTestPoint(event.location()))
     controller_->MouseClicked(event.x(), event.y());
 }
 
+void AutofillPopupViewViews::OnGestureEvent(ui::GestureEvent* event) {
+  if (!controller_)
+    return;
+
+  switch (event->type()) {
+    case ui::ET_GESTURE_TAP_DOWN:
+    case ui::ET_GESTURE_SCROLL_BEGIN:
+    case ui::ET_GESTURE_SCROLL_UPDATE:
+      if (HitTestPoint(event->location()))
+        controller_->MouseHovered(event->x(), event->y());
+      else
+        controller_->MouseExitedPopup();
+      break;
+    case ui::ET_GESTURE_TAP:
+    case ui::ET_GESTURE_SCROLL_END:
+      if (HitTestPoint(event->location()))
+        controller_->MouseClicked(event->x(), event->y());
+      else
+        controller_->MouseExitedPopup();
+      break;
+    case ui::ET_GESTURE_TAP_CANCEL:
+    case ui::ET_SCROLL_FLING_START:
+      controller_->MouseExitedPopup();
+      break;
+    default:
+      return;
+  }
+  event->SetHandled();

[Dan Beam, 2013/10/26 01:56:09]

^ this doesn't seem done

[mohsen, 2013/10/26 03:30:00]

non-handled events return from the function before reaching here (i.e. in the
'default' case of switch statement).

[Dan Beam, 2013/10/28 19:31:18]

ah, i see

+}
+
 void AutofillPopupViewViews::OnWidgetBoundsChanged(
     views::Widget* widget,
     const gfx::Rect& new_bounds) {
   DCHECK_EQ(widget, observing_widget_);
   controller_->Hide();
 }
 
 void AutofillPopupViewViews::Show() {
   if (!GetWidget()) {
     observing_widget_->AddObserver(this);
@@ skipping 98 matching lines @@
 
   // If the top level widget can't be found, cancel the popup since we can't
   // fully set it up.
   if (!observing_widget)
     return NULL;
 
   return new AutofillPopupViewViews(controller, observing_widget);
 }
 
 }  // namespace autofill