Commit de85750a authored by bulach@chromium.org's avatar bulach@chromium.org

Android: moves mojo to use long for JNI

BUG=317523

Review URL: https://codereview.chromium.org/131833002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@244163 0039d316-1c4b-4281-b951-d872f2087c98
parent 7f3a4077
...@@ -363,6 +363,7 @@ ...@@ -363,6 +363,7 @@
'type': 'none', 'type': 'none',
'variables': { 'variables': {
'jni_gen_package': 'mojo', 'jni_gen_package': 'mojo',
'jni_generator_ptr_type': 'long',
'input_java_class': 'java/util/HashSet.class', 'input_java_class': 'java/util/HashSet.class',
}, },
'includes': [ '../build/jar_file_jni_generator.gypi' ], 'includes': [ '../build/jar_file_jni_generator.gypi' ],
...@@ -378,7 +379,8 @@ ...@@ -378,7 +379,8 @@
'shell/android/apk/src/org/chromium/mojo_shell_apk/MojoMain.java', 'shell/android/apk/src/org/chromium/mojo_shell_apk/MojoMain.java',
], ],
'variables': { 'variables': {
'jni_gen_package': 'mojo' 'jni_gen_package': 'mojo',
'jni_generator_ptr_type': 'long',
}, },
'includes': [ '../build/jni_generator.gypi' ], 'includes': [ '../build/jni_generator.gypi' ],
}, },
......
...@@ -6,7 +6,6 @@ package org.chromium.mojo; ...@@ -6,7 +6,6 @@ package org.chromium.mojo;
import android.app.Activity; import android.app.Activity;
import android.content.Context; import android.content.Context;
import android.util.AttributeSet;
import android.view.MotionEvent; import android.view.MotionEvent;
import android.view.Surface; import android.view.Surface;
import android.view.SurfaceHolder; import android.view.SurfaceHolder;
...@@ -15,10 +14,13 @@ import android.view.SurfaceView; ...@@ -15,10 +14,13 @@ import android.view.SurfaceView;
import org.chromium.base.CalledByNative; import org.chromium.base.CalledByNative;
import org.chromium.base.JNINamespace; import org.chromium.base.JNINamespace;
/**
* Exposes SurfaceView to native code.
*/
@JNINamespace("mojo::services") @JNINamespace("mojo::services")
public class NativeViewportAndroid extends SurfaceView { public class NativeViewportAndroid extends SurfaceView {
private int mNativeMojoViewport; private long mNativeMojoViewport;
private final SurfaceHolder.Callback mSurfaceCallback; private final SurfaceHolder.Callback mSurfaceCallback;
@SuppressWarnings("unused") @SuppressWarnings("unused")
...@@ -27,7 +29,7 @@ public class NativeViewportAndroid extends SurfaceView { ...@@ -27,7 +29,7 @@ public class NativeViewportAndroid extends SurfaceView {
activity.setContentView(new NativeViewportAndroid(activity, nativeViewport)); activity.setContentView(new NativeViewportAndroid(activity, nativeViewport));
} }
public NativeViewportAndroid(Context context, int nativeViewport) { public NativeViewportAndroid(Context context, long nativeViewport) {
super(context); super(context);
mNativeMojoViewport = nativeViewport; mNativeMojoViewport = nativeViewport;
...@@ -72,16 +74,16 @@ public class NativeViewportAndroid extends SurfaceView { ...@@ -72,16 +74,16 @@ public class NativeViewportAndroid extends SurfaceView {
event.getEventTime()); event.getEventTime());
} }
private static native void nativeDestroy(int nativeNativeViewportAndroid); private static native void nativeDestroy(long nativeNativeViewportAndroid);
private static native void nativeSurfaceCreated( private static native void nativeSurfaceCreated(
int nativeNativeViewportAndroid, Surface surface); long nativeNativeViewportAndroid, Surface surface);
private static native void nativeSurfaceDestroyed( private static native void nativeSurfaceDestroyed(
int nativeNativeViewportAndroid); long nativeNativeViewportAndroid);
private static native void nativeSurfaceSetSize( private static native void nativeSurfaceSetSize(
int nativeNativeViewportAndroid, long nativeNativeViewportAndroid,
int width, int height); int width, int height);
private static native boolean nativeTouchEvent( private static native boolean nativeTouchEvent(
int nativeNativeViewportAndroid, long nativeNativeViewportAndroid,
int pointerId, int pointerId,
int action, int action,
float x, float y, float x, float y,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment