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

Unified Diff: chrome/browser/autofill/android/personal_data_manager_android.cc

Issue 436563005: Turn Autofill country text input into a Spinner. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 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: chrome/browser/autofill/android/personal_data_manager_android.cc
diff --git a/chrome/browser/autofill/android/personal_data_manager_android.cc b/chrome/browser/autofill/android/personal_data_manager_android.cc
index 57d7ec8cbce461027ce531397a69d8a424e44473..bd48309dd7b8e634de30bebbe107a74404e4c1b2 100644
--- a/chrome/browser/autofill/android/personal_data_manager_android.cc
+++ b/chrome/browser/autofill/android/personal_data_manager_android.cc
@@ -42,30 +42,20 @@ ScopedJavaLocalRef<jobject> CreateJavaProfileFromNative(
ConvertUTF16ToJavaString(env, profile.GetRawInfo(NAME_FULL)).obj(),
ConvertUTF16ToJavaString(env, profile.GetRawInfo(COMPANY_NAME)).obj(),
ConvertUTF16ToJavaString(
- env,
- profile.GetRawInfo(ADDRESS_HOME_STREET_ADDRESS)).obj(),
- ConvertUTF16ToJavaString(
- env,
- profile.GetRawInfo(ADDRESS_HOME_STATE)).obj(),
- ConvertUTF16ToJavaString(
- env,
- profile.GetRawInfo(ADDRESS_HOME_CITY)).obj(),
- ConvertUTF16ToJavaString(
- env,
- profile.GetRawInfo(ADDRESS_HOME_DEPENDENT_LOCALITY)).obj(),
- ConvertUTF16ToJavaString(
- env,
- profile.GetRawInfo(ADDRESS_HOME_ZIP)).obj(),
+ env, profile.GetRawInfo(ADDRESS_HOME_STREET_ADDRESS)).obj(),
+ ConvertUTF16ToJavaString(env, profile.GetRawInfo(ADDRESS_HOME_STATE))
+ .obj(),
+ ConvertUTF16ToJavaString(env, profile.GetRawInfo(ADDRESS_HOME_CITY))
+ .obj(),
ConvertUTF16ToJavaString(
- env,
- profile.GetRawInfo(ADDRESS_HOME_SORTING_CODE)).obj(),
- ConvertUTF16ToJavaString(
- env,
- profile.GetInfo(AutofillType(ADDRESS_HOME_COUNTRY),
- g_browser_process->GetApplicationLocale())).obj(),
+ env, profile.GetRawInfo(ADDRESS_HOME_DEPENDENT_LOCALITY)).obj(),
+ ConvertUTF16ToJavaString(env, profile.GetRawInfo(ADDRESS_HOME_ZIP)).obj(),
ConvertUTF16ToJavaString(
- env,
- profile.GetRawInfo(PHONE_HOME_WHOLE_NUMBER)).obj(),
+ env, profile.GetRawInfo(ADDRESS_HOME_SORTING_CODE)).obj(),
+ ConvertUTF16ToJavaString(env, profile.GetRawInfo(ADDRESS_HOME_COUNTRY))
+ .obj(),
+ ConvertUTF16ToJavaString(env, profile.GetRawInfo(PHONE_HOME_WHOLE_NUMBER))
+ .obj(),
ConvertUTF16ToJavaString(env, profile.GetRawInfo(EMAIL_ADDRESS)).obj(),
ConvertUTF8ToJavaString(env, profile.language_code()).obj());
}
@@ -117,11 +107,10 @@ void PopulateNativeProfileFromJava(
ConvertJavaStringToUTF16(
Java_AutofillProfile_getSortingCode(env, jprofile)),
g_browser_process->GetApplicationLocale());
- profile->SetInfo(
- AutofillType(ADDRESS_HOME_COUNTRY),
- ConvertJavaStringToUTF16(
- Java_AutofillProfile_getCountry(env, jprofile)),
- g_browser_process->GetApplicationLocale());
+ profile->SetInfo(AutofillType(ADDRESS_HOME_COUNTRY),
+ ConvertJavaStringToUTF16(
+ Java_AutofillProfile_getCountryCode(env, jprofile)),
+ g_browser_process->GetApplicationLocale());
profile->SetInfo(
AutofillType(PHONE_HOME_WHOLE_NUMBER),
ConvertJavaStringToUTF16(
@@ -175,6 +164,12 @@ void PopulateNativeCreditCardFromJava(
ConvertJavaStringToUTF16(Java_CreditCard_getYear(env, jcard)));
}
+/*
+ScopedJavaLocalRef<jobject> CreateJavaCountryFromNative(
aurimas (slooooooooow) 2014/08/01 15:18:21 Can you remove this?
Evan Stade 2014/08/01 18:59:23 oops. Yes.
Evan Stade 2014/08/04 20:36:04 Done.
+ JNIEnv* env,
+ const
+*/
+
} // namespace
PersonalDataManagerAndroid::PersonalDataManagerAndroid(JNIEnv* env,

Powered by Google App Engine
This is Rietveld 408576698