Disable double tap zoom on mobile sites, to remove 300ms click delay

content/public/android/javatests/src/org/chromium/content/browser/ContentViewGestureHandlerTest.java

 // 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.
 
 package org.chromium.content.browser;
 
 import android.content.Context;
 import android.os.Build;
 import android.os.Bundle;
 import android.os.SystemClock;
@@ skipping 1755 matching lines @@
         // If events are delivered at vsync, multiple SCROLL_BY and PINCH_BY events should still
         // trigger only a single vsync from any given call to confirmTouchEvent().
         if (inputEventsDeliveredAtVSync) {
             assertEquals(1, mMockMotionEventDelegate.mTotalSentLastGestureForVSyncCount);
         } else {
             assertEquals(0, mMockMotionEventDelegate.mTotalSentLastGestureForVSyncCount);
         }
    }
 
     /**
-     * Verify that no double tap gestures are created if the page has a fixed
-     * page scale.
+     * Verify that no double tap gestures are created if the gesture handler is
+     * told to disable double tap gesture detection (according to the logic in
+     * ContentViewCore.onRenderCoordinatesUpdated).
      * @throws Exception
      */
     @SmallTest
     @Feature({"Gestures"})
-    public void testNoDoubleTapWhenPageScaleFixed() throws Exception {
+    public void testNoDoubleTapWhenDoubleTapDisabled() throws Exception {
         final long downTime = SystemClock.uptimeMillis();
         final long eventTime = SystemClock.uptimeMillis();
 
         GestureRecordingMotionEventDelegate mockDelegate =
                 new GestureRecordingMotionEventDelegate();
         mGestureHandler = new ContentViewGestureHandler(
                 getInstrumentation().getTargetContext(), mockDelegate, mMockZoomManager,
                 ContentViewCore.INPUT_EVENTS_DELIVERED_AT_VSYNC);
-        mGestureHandler.updateHasFixedPageScale(true);
+        mGestureHandler.updateShouldDisableDoubleTap(true);
 
         MotionEvent event = MotionEvent.obtain(
                 downTime, downTime, MotionEvent.ACTION_DOWN,
                 FAKE_COORD_X, FAKE_COORD_Y, 0);
         assertTrue(mGestureHandler.onTouchEvent(event));
         assertEquals("No events should have been sent",
                 0, mockDelegate.mGestureTypeList.size());
 
         event = MotionEvent.obtain(
                 downTime, eventTime + 5, MotionEvent.ACTION_UP,
@@ skipping 18 matching lines @@
         assertTrue(mGestureHandler.onTouchEvent(event));
         assertEquals("A double tap should not have occurred",
                 ContentViewGestureHandler.GESTURE_SINGLE_TAP_CONFIRMED,
                 mockDelegate.mMostRecentGestureEvent.mType);
         assertEquals("Only GESTURE_SINGLE_TAP_CONFIRMED and " +
                 "GESTURE_SINGLE_TAP_CONFIRMED should have been sent",
                 2, mockDelegate.mGestureTypeList.size());
     }
 
     /**
-     * Verify that double tap drag zoom feature is not invoked
-     * when the page scale is fixed. The second tap sequence should be
-     * treated just as the first would be.
+     * Verify that double tap drag zoom feature is not invoked when the gesture
+     * handler is told to disable double tap gesture detection (according to the
+     * logic in ContentViewCore.onRenderCoordinatesUpdated).
+     * The second tap sequence should be treated just as the first would be.
      * @throws Exception
      */
     @SmallTest
     @Feature({"Gestures"})
-    public void testNoDoubleTapDragZoomWhenPageScaleFixed() throws Exception {
+    public void testNoDoubleTapDragZoomWhenDoubleTapDisabled() throws Exception {
         final long downTime1 = SystemClock.uptimeMillis();
         final long downTime2 = downTime1 + 100;
 
         GestureRecordingMotionEventDelegate mockDelegate =
                 new GestureRecordingMotionEventDelegate();
         mGestureHandler = new ContentViewGestureHandler(
                 getInstrumentation().getTargetContext(), mockDelegate, mMockZoomManager,
                 ContentViewCore.INPUT_EVENTS_DELIVERED_AT_VSYNC);
-        mGestureHandler.updateHasFixedPageScale(true);
+        mGestureHandler.updateShouldDisableDoubleTap(true);
 
         MotionEvent event = motionEvent(MotionEvent.ACTION_DOWN, downTime1, downTime1);
         assertTrue(mGestureHandler.onTouchEvent(event));
 
         event = MotionEvent.obtain(
                 downTime1, downTime1 + 5, MotionEvent.ACTION_UP,
                 FAKE_COORD_X, FAKE_COORD_Y, 0);
         mGestureHandler.onTouchEvent(event);
 
         event = MotionEvent.obtain(
@@ skipping 31 matching lines @@
 
         event = MotionEvent.obtain(
                 downTime2, downTime2 + 15, MotionEvent.ACTION_UP,
                 FAKE_COORD_X, FAKE_COORD_Y + 200, 0);
         assertTrue(mGestureHandler.onTouchEvent(event));
         assertFalse("No GESTURE_PINCH_END should have been sent",
                 mockDelegate.mGestureTypeList.contains(
                         ContentViewGestureHandler.GESTURE_PINCH_END));
     }
 }