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

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

Issue 2770613002: Forward GenericMotionEvent to EventForwarder (Closed)
Patch Set: comments Created 3 years, 9 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/input/JoystickScrollProvider.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/input/JoystickScrollProvider.java b/content/public/android/java/src/org/chromium/content/browser/input/JoystickScrollProvider.java
index 8c57b5f8f8531fb34f99ba8a4890ca492b78903d..d1eca2fd57f6fa528e65049ced8379745df2ecf6 100644
--- a/content/public/android/java/src/org/chromium/content/browser/input/JoystickScrollProvider.java
+++ b/content/public/android/java/src/org/chromium/content/browser/input/JoystickScrollProvider.java
@@ -12,9 +12,7 @@ import android.view.View;
import android.view.animation.AnimationUtils;
import org.chromium.base.Log;
-import org.chromium.base.annotations.CalledByNative;
-import org.chromium.base.annotations.JNINamespace;
-import org.chromium.content_public.browser.WebContents;
+import org.chromium.ui.base.EventForwarder;
import org.chromium.ui.base.WindowAndroid;
import org.chromium.ui.display.DisplayAndroid;
import org.chromium.ui.display.DisplayAndroid.DisplayAndroidObserver;
@@ -22,7 +20,6 @@ import org.chromium.ui.display.DisplayAndroid.DisplayAndroidObserver;
/**
* This class implements auto scrolling and panning for gamepad left joystick motion event.
*/
-@JNINamespace("content")
public class JoystickScrollProvider {
private static final String TAG = "JoystickScroll";
@@ -43,11 +40,11 @@ public class JoystickScrollProvider {
}
}
- private WindowAndroid mWindowAndroid;
- private View mContainerView;
- private long mNativeJoystickScrollProvider;
- private JoystickScrollDisplayObserver mDisplayObserver;
+ private final EventForwarder mEventForwarder;
+ private final JoystickScrollDisplayObserver mDisplayObserver;
+ private View mContainerView;
+ private WindowAndroid mWindowAndroid;
private float mScrollVelocityX;
private float mScrollVelocityY;
private float mScrollFactor;
@@ -63,20 +60,14 @@ public class JoystickScrollProvider {
* Constructs a new JoystickScrollProvider.
*/
public JoystickScrollProvider(
- WebContents webContents, View containerView, WindowAndroid windowAndroid) {
- mNativeJoystickScrollProvider = nativeInit(webContents);
+ View containerView, WindowAndroid windowAndroid, EventForwarder eventForwarder) {
mContainerView = containerView;
mWindowAndroid = windowAndroid;
+ mEventForwarder = eventForwarder;
mEnabled = true;
mDisplayObserver = new JoystickScrollDisplayObserver();
}
- @CalledByNative
- private void onNativeObjectDestroyed(long nativePointer) {
- assert nativePointer == mNativeJoystickScrollProvider;
- mNativeJoystickScrollProvider = 0;
- }
-
/**
* This function enables or disables scrolling through joystick.
* @param enabled Decides whether joystick scrolling should be
@@ -87,6 +78,10 @@ public class JoystickScrollProvider {
if (!enabled) stop();
}
+ public void setContainerView(View containerView) {
+ mContainerView = containerView;
+ }
+
public void onViewAttachedToWindow() {
addDisplayAndroidObserver();
}
@@ -118,12 +113,14 @@ public class JoystickScrollProvider {
Context context = mWindowAndroid == null ? null : mWindowAndroid.getContext().get();
TypedValue outValue = new TypedValue();
- if (context != null && context.getTheme().resolveAttribute(
- android.R.attr.listPreferredItemHeight, outValue, true)) {
+ if (context != null
+ && context.getTheme().resolveAttribute(
+ android.R.attr.listPreferredItemHeight, outValue, true)) {
mScrollFactor = outValue.getDimension(context.getResources().getDisplayMetrics());
} else {
if (context != null) {
- Log.d(TAG, "Theme attribute listPreferredItemHeight not defined"
+ Log.d(TAG,
+ "Theme attribute listPreferredItemHeight not defined"
+ " switching to fallback scroll factor");
}
mScrollFactor = SCROLL_FACTOR_FALLBACK * mDipScale;
@@ -134,6 +131,7 @@ public class JoystickScrollProvider {
* This function processes motion event and computes new
* scroll offest in pixels which is propertional to left joystick
* axes movement.
+ *
* It also starts runnable to scroll content view core equal to
* scroll offset pixels.
*
@@ -143,8 +141,9 @@ public class JoystickScrollProvider {
public boolean onMotion(MotionEvent event) {
if (!mEnabled) return false;
if ((event.getSource() & InputDevice.SOURCE_CLASS_JOYSTICK) == 0) return false;
- Log.d(TAG, "Joystick left stick axis: " + event.getAxisValue(MotionEvent.AXIS_X) + ","
- + event.getAxisValue(MotionEvent.AXIS_Y));
+ Log.d(TAG,
+ "Joystick left stick axis: " + event.getAxisValue(MotionEvent.AXIS_X) + ","
+ + event.getAxisValue(MotionEvent.AXIS_Y));
assert mScrollFactor != 0;
@@ -181,10 +180,7 @@ public class JoystickScrollProvider {
final float dx = (mScrollVelocityX * dt / 1000.f);
final float dy = (mScrollVelocityY * dt / 1000.f);
- if (mNativeJoystickScrollProvider != 0) {
- nativeScrollBy(
- mNativeJoystickScrollProvider, timeMillis, dx / mDipScale, dy / mDipScale);
- }
+ mEventForwarder.onMouseWheelEvent(timeMillis, 0, 0, -dx, -dy, 1.0f);
mLastAnimateTimeMillis = timeMillis;
mContainerView.postOnAnimation(mScrollRunnable);
@@ -197,7 +193,7 @@ public class JoystickScrollProvider {
/**
* Removes noise from joystick motion events.
*/
- private float getFilteredAxisValue(MotionEvent event, int axis) {
+ private static float getFilteredAxisValue(MotionEvent event, int axis) {
float axisValWithNoise = event.getAxisValue(axis);
if (axisValWithNoise > JOYSTICK_SCROLL_DEADZONE
|| axisValWithNoise < -JOYSTICK_SCROLL_DEADZONE) {
@@ -205,8 +201,4 @@ public class JoystickScrollProvider {
}
return 0f;
}
-
- private native long nativeInit(WebContents webContents);
- private native void nativeScrollBy(
- long nativeJoystickScrollProvider, long timeMs, float dxDip, float dyDip);
}

Powered by Google App Engine
This is Rietveld 408576698