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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java

Issue 14577008: Disable vsync on input for webview (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix test on ics Created 7 years, 7 months 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: content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
index 375645b46f85c16c9645227281c525b3fae7fe8c..4c1609c0fa63bc300f472d7d1dc90a5ad663b1f9 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
@@ -5,7 +5,6 @@
package org.chromium.content.browser;
import android.content.Context;
-import android.os.Build;
import android.os.Bundle;
import android.os.SystemClock;
import android.util.Log;
@@ -128,7 +127,7 @@ class ContentViewGestureHandler implements LongPressDelegate {
private float mAccumulatedScrollErrorY = 0;
// Whether input events are delivered right before vsync.
- private boolean mInputEventsDeliveredAtVSync = false;
+ private final boolean mInputEventsDeliveredAtVSync;
static final int GESTURE_SHOW_PRESSED_STATE = 0;
static final int GESTURE_DOUBLE_TAP = 1;
@@ -207,15 +206,15 @@ class ContentViewGestureHandler implements LongPressDelegate {
}
ContentViewGestureHandler(
- Context context, MotionEventDelegate delegate, ZoomManager zoomManager) {
+ Context context, MotionEventDelegate delegate, ZoomManager zoomManager,
+ int inputEventDeliveryMode) {
mExtraParamBundle = new Bundle();
mLongPressDetector = new LongPressDetector(context, this);
mMotionEventDelegate = delegate;
mZoomManager = zoomManager;
mSnapScrollController = new SnapScrollController(context, mZoomManager);
-
- // Input events are delivered at vsync time on JB+.
- mInputEventsDeliveredAtVSync = (Build.VERSION.SDK_INT >= Build.VERSION_CODES.JELLY_BEAN);
+ mInputEventsDeliveredAtVSync =
+ inputEventDeliveryMode == ContentViewCore.INPUT_EVENTS_DELIVERED_AT_VSYNC;
initGestureDetectors(context);
}

Powered by Google App Engine
This is Rietveld 408576698