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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java

Issue 67573003: Android: moves chrome/ 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: chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
index d387c1f892db2dcb386dc9d79477b91f15b4e2dd..b929a18bb06e456e8811f4069f1bbacbf9c2d2a9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
@@ -25,7 +25,7 @@ public class FaviconHelper {
public static final int TOUCH_ICON = 1 << 1;
public static final int TOUCH_PRECOMPOSED_ICON = 1 << 2;
- private int mNativeFaviconHelper;
+ private long mNativeFaviconHelper;
/**
* Callback interface for getting the result from getLocalFaviconImageForURL method.
@@ -104,13 +104,13 @@ public class FaviconHelper {
return nativeGetSyncedFaviconImageForURL(mNativeFaviconHelper, profile, pageUrl);
}
- private static native int nativeInit();
- private static native void nativeDestroy(int nativeFaviconHelper);
- private static native boolean nativeGetLocalFaviconImageForURL(int nativeFaviconHelper,
+ private static native long nativeInit();
+ private static native void nativeDestroy(long nativeFaviconHelper);
+ private static native boolean nativeGetLocalFaviconImageForURL(long nativeFaviconHelper,
Profile profile, String pageUrl, int iconTypes, int desiredSizeInDip,
FaviconImageCallback faviconImageCallback);
- private static native Bitmap nativeGetSyncedFaviconImageForURL(int nativeFaviconHelper,
+ private static native Bitmap nativeGetSyncedFaviconImageForURL(long nativeFaviconHelper,
Profile profile, String pageUrl);
- private static native int nativeGetDominantColorForBitmap(int nativeFaviconHelper,
+ private static native int nativeGetDominantColorForBitmap(long nativeFaviconHelper,
Bitmap image);
}

Powered by Google App Engine
This is Rietveld 408576698