[Android] Provide unhandled tap event notifications

content/browser/web_contents/touch_editable_impl_aura_browsertest.cc

Index: content/browser/web_contents/touch_editable_impl_aura_browsertest.cc
diff --git a/content/browser/web_contents/touch_editable_impl_aura_browsertest.cc b/content/browser/web_contents/touch_editable_impl_aura_browsertest.cc
index 2a42dda5ccab1e2b6aab75e92cd4fafcf162063f..00b046bf243293ecf1f7cbc97619c461283bf1da 100644
--- a/content/browser/web_contents/touch_editable_impl_aura_browsertest.cc
+++ b/content/browser/web_contents/touch_editable_impl_aura_browsertest.cc
@@ -11,6 +11,7 @@
 #include "base/values.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/browser/web_contents/web_contents_view_aura.h"
+#include "content/common/input/web_input_event_traits.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/web_contents_view.h"
 #include "content/public/common/content_switches.h"
@@ -324,9 +325,14 @@ class TouchEditableImplAuraTest : public ContentBrowserTest {
     // Tap textfield
     touch_editable->Reset();
     generator.GestureTapAt(gfx::Point(bounds.x() + 50, bounds.y() + 40));
-    // Tap Down and Tap acks are sent synchronously.
-    touch_editable->WaitForSelectionChangeCallback();
-    touch_editable->Reset();
+    if (WebInputEventTraits::IgnoresAckDisposition(
+            blink::WebInputEvent::GestureTap)) {

[jdduke (slow), 2014/04/04 03:26:53]

tdresser@: What am I missing here?  Looks like this test is still failing...

[tdresser, 2014/04/04 13:28:19]

Hmmm, I can't repro the failure locally, with a "use_aura=1 use_ash=1
chromeos=1" build... I'm really not sure what's going on here.


Are you positive that waiting for the ack waits long enough? Do you need to:

touch_editable->Reset();
touch_editable->WaitForGestureAck();
touch_editable->WaitForSelectionChangeCallback();

+      touch_editable->WaitForSelectionChangeCallback();
+      touch_editable->Reset();
+    } else {
+      touch_editable->Reset();
+      touch_editable->WaitForGestureAck();
+    }
 
     // Check if cursor handle is showing.
     ui::TouchSelectionController* controller =