[WebPayments] Show labels on incomplete profiles

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

 // Copyright 2017 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/ui/views/payments/profile_list_view_controller.h"
 
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
 #include "chrome/browser/ui/views/payments/payment_request_row_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_views_util.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/payments/content/payment_request_spec.h"
 #include "components/payments/content/payment_request_state.h"
+#include "components/payments/core/profile_util.h"
 #include "components/payments/core/strings_util.h"
 #include "components/strings/grit/components_strings.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/controls/button/label_button.h"
 #include "ui/views/controls/button/md_text_button.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/layout/grid_layout.h"
 
@@ skipping 44 matching lines @@
   }
 
   void SelectedStateChanged() override {
     if (selected()) {
       parent_view_->SelectProfile(profile_);
       dialog_->GoBack();
     }
   }
 
   bool CanBeSelected() const override {
-    // TODO(crbug.com/709454): Check for profile completeness. Currently a giant
-    // hack to test the UI.
-    return !profile_->GetRawInfo(autofill::ADDRESS_HOME_ZIP).empty();
+    return parent_view_->ValidateProfile(profile_);
   }
 
   void PerformSelectionFallback() override {
     parent_view_->ShowEditor(profile_);
   }
 
   ProfileListViewController* parent_view_;
   autofill::AutofillProfile* profile_;
   PaymentRequestDialogView* dialog_;
 
@@ skipping 10 matching lines @@
       : ProfileListViewController(spec, state, dialog) {
     PopulateList();
   }
   ~ShippingProfileViewController() override {}
 
  protected:
   // ProfileListViewController:
   std::unique_ptr<views::View> GetLabel(
       autofill::AutofillProfile* profile) override {
     return GetShippingAddressLabel(AddressStyleType::DETAILED,
-                                   state()->GetApplicationLocale(), *profile);
+                                   state()->GetApplicationLocale(), *profile,
+                                   *spec(), *(state()->profile_comparator()));
   }
 
   void SelectProfile(autofill::AutofillProfile* profile) override {
     state()->SetSelectedShippingProfile(profile);
   }
 
   void ShowEditor(autofill::AutofillProfile* profile) override {
     dialog()->ShowShippingAddressEditor(
         /*on_edited=*/base::BindOnce(
             &PaymentRequestState::SetSelectedShippingProfile,
             base::Unretained(state()), profile),
         /*on_added=*/
         base::BindOnce(&PaymentRequestState::AddAutofillShippingProfile,
                        base::Unretained(state()), /*selected=*/true),
         profile);
   }
 
   autofill::AutofillProfile* GetSelectedProfile() override {
     return state()->selected_shipping_profile();
   }
 
+  bool ValidateProfile(autofill::AutofillProfile* profile) override {
+    return state()->profile_comparator()->IsShippingComplete(profile);
+  }
+
   std::vector<autofill::AutofillProfile*> GetProfiles() override {
     return state()->shipping_profiles();
   }
 
   DialogViewID GetDialogViewId() override {
     return DialogViewID::SHIPPING_ADDRESS_SHEET_LIST_VIEW;
   }
 
   base::string16 GetSheetTitle() override {
     return GetShippingAddressSectionString(spec()->shipping_type());
@@ skipping 25 matching lines @@
     PopulateList();
   }
   ~ContactProfileViewController() override {}
 
  protected:
   // ProfileListViewController:
   std::unique_ptr<views::View> GetLabel(
       autofill::AutofillProfile* profile) override {
     return GetContactInfoLabel(AddressStyleType::DETAILED,
                                state()->GetApplicationLocale(), *profile,
-                               *spec());
+                               *spec(), *(state()->profile_comparator()));
   }
 
   void SelectProfile(autofill::AutofillProfile* profile) override {
     state()->SetSelectedContactProfile(profile);
   }
 
   void ShowEditor(autofill::AutofillProfile* profile) override {
     dialog()->ShowContactInfoEditor(profile);
   }
 
   autofill::AutofillProfile* GetSelectedProfile() override {
     return state()->selected_contact_profile();
   }
 
+  bool ValidateProfile(autofill::AutofillProfile* profile) override {
+    return state()->profile_comparator()->IsContactInfoComplete(profile);
+  }
+
   std::vector<autofill::AutofillProfile*> GetProfiles() override {
     return state()->contact_profiles();
   }
 
   DialogViewID GetDialogViewId() override {
     return DialogViewID::CONTACT_INFO_SHEET_LIST_VIEW;
   }
 
   base::string16 GetSheetTitle() override {
     return l10n_util::GetStringUTF16(
@@ skipping 82 matching lines @@
 
 void ProfileListViewController::ButtonPressed(views::Button* sender,
                                               const ui::Event& event) {
   if (sender->tag() == GetSecondaryButtonTag())
     ShowEditor(nullptr);
   else
     PaymentRequestSheetController::ButtonPressed(sender, event);
 }
 
 }  // namespace payments