[WebPayments] Show labels on incomplete profiles

chrome/browser/ui/views/payments/profile_list_view_controller.h

Index: chrome/browser/ui/views/payments/profile_list_view_controller.h
diff --git a/chrome/browser/ui/views/payments/profile_list_view_controller.h b/chrome/browser/ui/views/payments/profile_list_view_controller.h
index 7e1af22e9233a1340406a004277d6d40ec1db101..b9fec775cbeddafdf5104b61a7e0b522628cd863 100644
--- a/chrome/browser/ui/views/payments/profile_list_view_controller.h
+++ b/chrome/browser/ui/views/payments/profile_list_view_controller.h
@@ -65,6 +65,8 @@ class ProfileListViewController : public PaymentRequestSheetController {
 
   virtual autofill::AutofillProfile* GetSelectedProfile() = 0;
 
+  virtual bool ValidateProfile(autofill::AutofillProfile* profile) = 0;

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

can this be const ref?

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

If it doesn't have side-effects, perhaps IsValidProfile is a better wording?
ValidateProfile feels like it's going to do all sorts of things to the |profile|

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

Done to both

+
  protected:
   // Does not take ownership of the arguments, which should outlive this object.
   ProfileListViewController(PaymentRequestSpec* spec,