Make autofill's Address store country using the country code so that app locale isn't needed for th…

chrome/browser/autofill/android/personal_data_manager_android.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/autofill/android/personal_data_manager_android.h"
 
 #include "base/android/jni_string.h"
 #include "base/format_macros.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autofill/personal_data_manager_factory.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_manager.h"
+#include "components/autofill/browser/autofill_country.h"

[Ilya Sherman, 2013/04/05 07:18:41]

nit: No longer needed.

[jam, 2013/04/05 07:35:35]

Done.

 #include "components/autofill/browser/field_types.h"
 #include "components/autofill/browser/personal_data_manager.h"
 #include "jni/PersonalDataManager_jni.h"
 
 using base::android::ConvertJavaStringToUTF8;
 using base::android::ConvertUTF16ToJavaString;
 using base::android::ConvertUTF8ToJavaString;
 using base::android::ScopedJavaLocalRef;
 
 namespace {
@@ skipping 20 matching lines @@
           env,
           profile.GetRawInfo(ADDRESS_HOME_CITY)).obj(),
       ConvertUTF16ToJavaString(
           env,
           profile.GetRawInfo(ADDRESS_HOME_STATE)).obj(),
       ConvertUTF16ToJavaString(
           env,
           profile.GetRawInfo(ADDRESS_HOME_ZIP)).obj(),
       ConvertUTF16ToJavaString(
           env,
-          profile.GetRawInfo(ADDRESS_HOME_COUNTRY)).obj(),
+          profile.GetInfo(ADDRESS_HOME_COUNTRY,
+                          g_browser_process->GetApplicationLocale())).obj(),
       ConvertUTF16ToJavaString(
           env,
           profile.GetRawInfo(PHONE_HOME_WHOLE_NUMBER)).obj(),
       ConvertUTF16ToJavaString(env, profile.GetRawInfo(EMAIL_ADDRESS)).obj());
 }
 
 void PopulateNativeProfileFromJava(
     const jobject& jprofile,
     JNIEnv* env,
     AutofillProfile* profile) {
@@ skipping 18 matching lines @@
       ConvertJavaStringToUTF16(
           Java_AutofillProfile_getCity(env, jprofile)));
   profile->SetRawInfo(
       ADDRESS_HOME_STATE,
       ConvertJavaStringToUTF16(
           Java_AutofillProfile_getState(env, jprofile)));
   profile->SetRawInfo(
       ADDRESS_HOME_ZIP,
       ConvertJavaStringToUTF16(
           Java_AutofillProfile_getZip(env, jprofile)));
-  profile->SetRawInfo(
+  profile->SetInfo(
       ADDRESS_HOME_COUNTRY,
       ConvertJavaStringToUTF16(
-          Java_AutofillProfile_getCountry(env, jprofile)));
+          Java_AutofillProfile_getCountry(env, jprofile)),
+          g_browser_process->GetApplicationLocale());

[Ilya Sherman, 2013/04/05 07:18:41]

nit: Looks like this line should be indented four fewer spaces.

[jam, 2013/04/05 07:35:35]

Done.

   profile->SetRawInfo(
       PHONE_HOME_WHOLE_NUMBER,
       ConvertJavaStringToUTF16(
           Java_AutofillProfile_getPhoneNumber(env, jprofile)));
   profile->SetRawInfo(
       EMAIL_ADDRESS,
       ConvertJavaStringToUTF16(
           Java_AutofillProfile_getEmailAddress(env, jprofile)));
 }
 
@@ skipping 163 matching lines @@
 // static
 bool PersonalDataManagerAndroid::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 static jint Init(JNIEnv* env, jobject obj) {
   PersonalDataManagerAndroid* personal_data_manager_android =
       new PersonalDataManagerAndroid(env, obj);
   return reinterpret_cast<jint>(personal_data_manager_android);
 }