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

Unified Diff: sync/util/session_utils_android.cc

Issue 11038015: Android: lazy initialization for method id. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 8 years, 2 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
« no previous file with comments | « net/proxy/proxy_config_service_android.cc ('k') | ui/base/clipboard/clipboard_android.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/util/session_utils_android.cc
diff --git a/sync/util/session_utils_android.cc b/sync/util/session_utils_android.cc
index 69f1ae448a2581f4b7e341674ee51362778d3248..7be415d1b771cf8ab22476b8e458e393c4415e3e 100644
--- a/sync/util/session_utils_android.cc
+++ b/sync/util/session_utils_android.cc
@@ -17,6 +17,7 @@ using base::android::CheckException;
using base::android::ConvertUTF8ToJavaString;
using base::android::GetApplicationContext;
using base::android::GetClass;
+using base::android::MethodID;
using base::android::JavaRef;
using base::android::ScopedJavaLocalRef;
@@ -26,9 +27,10 @@ ScopedJavaLocalRef<jstring> GetAndroidIdJNI(
JNIEnv* env, const JavaRef<jobject>& content_resolver) {
ScopedJavaLocalRef<jclass> clazz(
GetClass(env, "android/provider/Settings$Secure"));
- jmethodID j_get_string = GetStaticMethodID(env, clazz, "getString",
+ jmethodID j_get_string = MethodID::Get<MethodID::TYPE_STATIC>(
+ env, clazz.obj(), "getString",
"(Landroid/content/ContentResolver;Ljava/lang/String;)"
- "Ljava/lang/String;");
+ "Ljava/lang/String;");
ScopedJavaLocalRef<jstring> j_android_id =
ConvertUTF8ToJavaString(env, "android_id");
jstring android_id = static_cast<jstring>(
@@ -41,8 +43,10 @@ ScopedJavaLocalRef<jstring> GetAndroidIdJNI(
ScopedJavaLocalRef<jobject> GetContentResolver(JNIEnv* env) {
ScopedJavaLocalRef<jclass> clazz(GetClass(env, "android/content/Context"));
- jmethodID j_get_content_resolver_method = GetMethodID(
- env, clazz,"getContentResolver", "()Landroid/content/ContentResolver;");
+ jmethodID j_get_content_resolver_method = MethodID::Get<
+ MethodID::TYPE_INSTANCE>(
+ env, clazz.obj(), "getContentResolver",
+ "()Landroid/content/ContentResolver;");
jobject content_resolver = env->CallObjectMethod(
GetApplicationContext(), j_get_content_resolver_method);
CheckException(env);
« no previous file with comments | « net/proxy/proxy_config_service_android.cc ('k') | ui/base/clipboard/clipboard_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698