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

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

Issue 59673007: Android: moves content/ to use long for JNI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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/ContentSettings.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentSettings.java b/content/public/android/java/src/org/chromium/content/browser/ContentSettings.java
index d3ca0c6b52f7cb4563b922b88779c49dc2091a5b..3a63ba8e13c7520551cbfa9bdce408d0b13750cf 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentSettings.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentSettings.java
@@ -19,7 +19,7 @@ public class ContentSettings {
// The native side of this object. Ownership is retained native-side by the WebContents
// instance that backs the associated ContentViewCore.
- private int mNativeContentSettings = 0;
+ private long mNativeContentSettings = 0;
private ContentViewCore mContentViewCore;
@@ -27,7 +27,7 @@ public class ContentSettings {
* Package constructor to prevent clients from creating a new settings
* instance. Must be called on the UI thread.
*/
- ContentSettings(ContentViewCore contentViewCore, int nativeContentView) {
+ ContentSettings(ContentViewCore contentViewCore, long nativeContentView) {
ThreadUtils.assertOnUiThread();
mContentViewCore = contentViewCore;
mNativeContentSettings = nativeInit(nativeContentView);
@@ -56,7 +56,7 @@ public class ContentSettings {
}
// Initialize the ContentSettings native side.
- private native int nativeInit(int contentViewPtr);
+ private native long nativeInit(long contentViewPtr);
- private native boolean nativeGetJavaScriptEnabled(int nativeContentSettings);
+ private native boolean nativeGetJavaScriptEnabled(long nativeContentSettings);
}

Powered by Google App Engine
This is Rietveld 408576698