[WebPayments] Show labels on incomplete profiles

components/autofill/core/browser/autofill_profile_comparator.h

Index: components/autofill/core/browser/autofill_profile_comparator.h
diff --git a/components/autofill/core/browser/autofill_profile_comparator.h b/components/autofill/core/browser/autofill_profile_comparator.h
index 4b69d184112b18d5d5d46207e00807efd96fcecb..d8ca6055ee76675d487ffd13619d7a792127e7cf 100644
--- a/components/autofill/core/browser/autofill_profile_comparator.h
+++ b/components/autofill/core/browser/autofill_profile_comparator.h
@@ -215,10 +215,11 @@ class AutofillProfileComparator {
                      const AutofillProfile& p2,
                      NameInfo* info) const;
 
+  const std::string app_locale_;

[Mathieu, 2017/04/27 18:34:01]

hmm, I think the style is to have a getter() that's visible and app_locale_
stays private

[Mathieu, 2017/04/27 18:34:30]

Also do you need it, can you get it another way?

[tmartino, 2017/04/28 17:10:06]

We take it in at construct-time and pass it to the AutofillProfileComparator. I
guess the alternative would be to store a copy at the subclass level, but that
seems silly. Added a getter.

+
  private:
   l10n::CaseInsensitiveCompare case_insensitive_compare_;
   std::unique_ptr<icu::Transliterator> transliterator_;
-  const std::string app_locale_;
 
   DISALLOW_COPY_AND_ASSIGN(AutofillProfileComparator);
 };