[Payments] Don't show Organization in billing address selection label.

chrome/browser/autofill/android/personal_data_manager_android.cc

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 2c739bf29ba0a1e651191c1d1e814955af14c6e7..f031ddc629ac13db26d5af1eeaa0962f2f722606 100644
--- a/chrome/browser/autofill/android/personal_data_manager_android.cc
+++ b/chrome/browser/autofill/android/personal_data_manager_android.cc
@@ -388,7 +388,7 @@ ScopedJavaLocalRef<jobjectArray>
 PersonalDataManagerAndroid::GetProfileLabelsForSettings(
     JNIEnv* env,
     const JavaParamRef<jobject>& unused_obj) {
-  return GetProfileLabels(env, false, false,
+  return GetProfileLabels(env, false, false, true,

[please use gerrit instead, 2016/11/04 21:27:32]

Please add a comment here about the meaning of each of these three boolean
parameters. Unfortunately, a list of boolean parameters is a common anti-pattern
that makes reading code harder. Comments is one way to fight the anti-pattern,
although not the most ideal, because the compiler cannot check your comments for
correctness. In the interest of time for M56 release, let's just use comments.

[sebsg, 2016/11/07 15:53:02]

Done.

                           personal_data_manager_->GetProfiles());
 }
 
@@ -396,8 +396,10 @@ ScopedJavaLocalRef<jobjectArray>
 PersonalDataManagerAndroid::GetProfileLabelsToSuggest(
     JNIEnv* env,
     const JavaParamRef<jobject>& unused_obj,
-    jboolean include_name_in_label) {
+    jboolean include_name_in_label,
+    jboolean include_organization_in_label) {
   return GetProfileLabels(env, true, include_name_in_label,

[please use gerrit instead, 2016/11/04 21:27:32]

Please add a comment about the meaning of this "true" boolean parameter.

[sebsg, 2016/11/07 15:53:01]

Done.

+                          include_organization_in_label,
                           personal_data_manager_->GetProfilesToSuggest());
 }
 
@@ -770,6 +772,7 @@ ScopedJavaLocalRef<jobjectArray> PersonalDataManagerAndroid::GetProfileLabels(
     JNIEnv* env,
     bool address_only,
     bool include_name_in_label,
+    bool include_organization_in_label,
     std::vector<AutofillProfile*> profiles) {
   std::unique_ptr<std::vector<ServerFieldType>> suggested_fields;
   size_t minimal_fields_shown = 2;
@@ -777,7 +780,8 @@ ScopedJavaLocalRef<jobjectArray> PersonalDataManagerAndroid::GetProfileLabels(
     suggested_fields.reset(new std::vector<ServerFieldType>);
     if (include_name_in_label)
       suggested_fields->push_back(NAME_FULL);
-    suggested_fields->push_back(COMPANY_NAME);
+    if (include_organization_in_label)
+      suggested_fields->push_back(COMPANY_NAME);
     suggested_fields->push_back(ADDRESS_HOME_LINE1);
     suggested_fields->push_back(ADDRESS_HOME_LINE2);
     suggested_fields->push_back(ADDRESS_HOME_DEPENDENT_LOCALITY);