[Payments] Display the shipping option error alongside the address

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

 // Copyright 2016 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/payment_request_views_util.h"
 
 #include <vector>
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
@@ skipping 48 matching lines @@
   fields.push_back(autofill::ADDRESS_HOME_DEPENDENT_LOCALITY);
   fields.push_back(autofill::ADDRESS_HOME_CITY);
   fields.push_back(autofill::ADDRESS_HOME_STATE);
   fields.push_back(autofill::ADDRESS_HOME_ZIP);
   fields.push_back(autofill::ADDRESS_HOME_SORTING_CODE);
 
   return profile.ConstructInferredLabel(fields, fields.size(), locale);
 }
 
 // |s1|, |s2|, and |s3| are lines identifying the profile. |s1| is the
-// "headline" which may be emphasized depending on |type|. |error| is a
-// message indicating errors that need to be resolved before using this
-// profile.
-std::unique_ptr<views::View> GetProfileLabel(AddressStyleType type,
-                                             const base::string16& s1,
-                                             const base::string16& s2,
-                                             const base::string16& s3,
-                                             const base::string16& error) {
+// "headline" which may be emphasized depending on |type|. If |disabled_state|
+// is true, the labels will look disabled.
+std::unique_ptr<views::View> GetBaseProfileLabel(AddressStyleType type,
+                                                 const base::string16& s1,
+                                                 const base::string16& s2,
+                                                 const base::string16& s3,
+                                                 bool disabled_state = false) {
   std::unique_ptr<views::View> container = base::MakeUnique<views::View>();
   std::unique_ptr<views::BoxLayout> layout =
       base::MakeUnique<views::BoxLayout>(views::BoxLayout::kVertical, 0, 0, 0);
   layout->set_cross_axis_alignment(
       views::BoxLayout::CROSS_AXIS_ALIGNMENT_START);
   container->SetLayoutManager(layout.release());
 
   if (!s1.empty()) {
     std::unique_ptr<views::Label> label = base::MakeUnique<views::Label>(s1);
     if (type == AddressStyleType::DETAILED) {
       const gfx::FontList& font_list = label->font_list();
       label->SetFontList(font_list.DeriveWithWeight(gfx::Font::Weight::BOLD));
     }
     label->set_id(static_cast<int>(DialogViewID::PROFILE_LABEL_LINE_1));
     label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
+    if (disabled_state) {
+      label->SetEnabledColor(label->GetNativeTheme()->GetSystemColor(
+          ui::NativeTheme::kColorId_LabelDisabledColor));
+    }
     container->AddChildView(label.release());
   }
 
   if (!s2.empty()) {
     std::unique_ptr<views::Label> label = base::MakeUnique<views::Label>(s2);
     label->set_id(static_cast<int>(DialogViewID::PROFILE_LABEL_LINE_2));
     label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
+    if (disabled_state) {
+      label->SetEnabledColor(label->GetNativeTheme()->GetSystemColor(
+          ui::NativeTheme::kColorId_LabelDisabledColor));
+    }
     container->AddChildView(label.release());
   }
 
   if (!s3.empty()) {
     std::unique_ptr<views::Label> label = base::MakeUnique<views::Label>(s3);
     label->set_id(static_cast<int>(DialogViewID::PROFILE_LABEL_LINE_3));
     label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
+    if (disabled_state) {
+      label->SetEnabledColor(label->GetNativeTheme()->GetSystemColor(
+          ui::NativeTheme::kColorId_LabelDisabledColor));
+    }
     container->AddChildView(label.release());
   }
-
-  if (!error.empty()) {
-    std::unique_ptr<views::Label> label = base::MakeUnique<views::Label>(error);
-    label->set_id(static_cast<int>(DialogViewID::PROFILE_LABEL_ERROR));
-    label->SetFontList(label->GetDefaultFontList().DeriveWithSizeDelta(-1));
-    label->SetEnabledColor(label->GetNativeTheme()->GetSystemColor(
-        ui::NativeTheme::kColorId_LinkEnabled));
-    container->AddChildView(label.release());
-  }
-
   return container;
 }
 
+// Returns a label representing the |profile| as a shipping address. See
+// GetBaseProfileLabel() for more documentation.
+std::unique_ptr<views::View> GetShippingAddressLabel(
+    AddressStyleType type,
+    const std::string& locale,
+    const autofill::AutofillProfile& profile,
+    bool disabled_state) {
+  base::string16 name =
+      profile.GetInfo(autofill::AutofillType(autofill::NAME_FULL), locale);
+
+  base::string16 address = GetAddressFromProfile(profile, locale);
+
+  base::string16 phone = profile.GetInfo(
+      autofill::AutofillType(autofill::PHONE_HOME_WHOLE_NUMBER), locale);
+
+  return GetBaseProfileLabel(type, name, address, phone, disabled_state);
+}
+
 // Paints the gray horizontal line that doesn't span the entire width of the
 // dialog at the bottom of the view it borders.
 class PaymentRequestRowBorderPainter : public views::Painter {
  public:
   PaymentRequestRowBorderPainter() {}
   ~PaymentRequestRowBorderPainter() override {}
 
   // views::Painter:
   gfx::Size GetMinimumSize() const override {
     return gfx::Size(2 * payments::kPaymentRequestRowHorizontalInsets, 1);
@@ skipping 105 matching lines @@
   chrome_logo->set_can_process_events_within_subtree(false);
   chrome_logo->SetImage(ResourceBundle::GetSharedInstance()
                             .GetImageNamed(IDR_PRODUCT_LOGO_NAME_22)
                             .AsImageSkia());
   chrome_logo->SetTooltipText(l10n_util::GetStringUTF16(IDS_PRODUCT_NAME));
   content_view->AddChildView(chrome_logo.release());
 
   return content_view;
 }
 
-std::unique_ptr<views::View> GetShippingAddressLabel(
+std::unique_ptr<views::View> GetShippingAddressLabelWithError(
     AddressStyleType type,
     const std::string& locale,
     const autofill::AutofillProfile& profile,
-    const PaymentOptionsProvider& options,
+    const std::string& error,
+    bool disabled_state) {
+  std::unique_ptr<views::View> base_label =
+      GetShippingAddressLabel(type, locale, profile, disabled_state);
+
+  if (!error.empty()) {
+    std::unique_ptr<views::Label> label =
+        base::MakeUnique<views::Label>(base::UTF8ToUTF16(error));
+    label->set_id(static_cast<int>(DialogViewID::PROFILE_LABEL_ERROR));
+    label->SetFontList(label->GetDefaultFontList().DeriveWithSizeDelta(-1));
+    // Error information is typically in red.
+    label->SetEnabledColor(label->GetNativeTheme()->GetSystemColor(
+        ui::NativeTheme::kColorId_AlertSeverityHigh));
+    base_label->AddChildView(label.release());
+  }
+  return base_label;
+}
+
+std::unique_ptr<views::View> GetShippingAddressLabelWithMissingInfo(
+    AddressStyleType type,
+    const std::string& locale,
+    const autofill::AutofillProfile& profile,
     const PaymentsProfileComparator& comp) {
-  base::string16 name =
-      profile.GetInfo(autofill::AutofillType(autofill::NAME_FULL), locale);
+  std::unique_ptr<views::View> base_label =
+      GetShippingAddressLabel(type, locale, profile, /*disabled_state=*/false);
 
-  base::string16 address = GetAddressFromProfile(profile, locale);
-
-  base::string16 phone = profile.GetInfo(
-      autofill::AutofillType(autofill::PHONE_HOME_WHOLE_NUMBER), locale);
-
-  base::string16 error = comp.GetStringForMissingShippingFields(profile);
-
-  return GetProfileLabel(type, name, address, phone, error);
+  base::string16 missing = comp.GetStringForMissingShippingFields(profile);
+  if (!missing.empty()) {
+    std::unique_ptr<views::Label> label =
+        base::MakeUnique<views::Label>(missing);
+    label->set_id(static_cast<int>(DialogViewID::PROFILE_LABEL_ERROR));
+    label->SetFontList(label->GetDefaultFontList().DeriveWithSizeDelta(-1));
+    // Missing information typically has a nice shade of blue.
+    label->SetEnabledColor(label->GetNativeTheme()->GetSystemColor(
+        ui::NativeTheme::kColorId_LinkEnabled));
+    base_label->AddChildView(label.release());
+  }
+  return base_label;
 }
 
 // TODO(anthonyvd): unit test the label layout.
 std::unique_ptr<views::View> GetContactInfoLabel(
     AddressStyleType type,
     const std::string& locale,
     const autofill::AutofillProfile& profile,
     const PaymentOptionsProvider& options,
     const PaymentsProfileComparator& comp) {
   base::string16 name =
       options.request_payer_name()
           ? profile.GetInfo(autofill::AutofillType(autofill::NAME_FULL), locale)
           : base::string16();
 
   base::string16 phone =
       options.request_payer_phone()
           ? profile.GetInfo(
                 autofill::AutofillType(autofill::PHONE_HOME_WHOLE_NUMBER),
                 locale)
           : base::string16();
 
   base::string16 email =
       options.request_payer_email()
           ? profile.GetInfo(autofill::AutofillType(autofill::EMAIL_ADDRESS),
                             locale)
           : base::string16();
 
-  base::string16 error = comp.GetStringForMissingContactFields(profile);
+  std::unique_ptr<views::View> base_label =
+      GetBaseProfileLabel(type, name, phone, email);
 
-  return GetProfileLabel(type, name, phone, email, error);
+  base::string16 missing = comp.GetStringForMissingContactFields(profile);
+  if (!missing.empty()) {

[anthonyvd, 2017/05/04 13:07:37]

This entire block is the same as the one on line 296, can we make it a function?

[Mathieu, 2017/05/04 13:53:25]

Done.

+    std::unique_ptr<views::Label> label =
+        base::MakeUnique<views::Label>(missing);
+    label->set_id(static_cast<int>(DialogViewID::PROFILE_LABEL_ERROR));
+    label->SetFontList(label->GetDefaultFontList().DeriveWithSizeDelta(-1));
+    // Missing information typically has a nice shade of blue.
+    label->SetEnabledColor(label->GetNativeTheme()->GetSystemColor(
+        ui::NativeTheme::kColorId_LinkEnabled));
+    base_label->AddChildView(label.release());
+  }
+  return base_label;
 }
 
 std::unique_ptr<views::Border> CreatePaymentRequestRowBorder() {
   return views::CreateBorderPainter(
       base::MakeUnique<PaymentRequestRowBorderPainter>(),
       gfx::Insets());
 }
 
 std::unique_ptr<views::Label> CreateBoldLabel(const base::string16& text) {
   std::unique_ptr<views::Label> label = base::MakeUnique<views::Label>(text);
@@ skipping 29 matching lines @@
     amount_label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
     amount_label->set_id(
         static_cast<int>(DialogViewID::SHIPPING_OPTION_AMOUNT));
     container->AddChildView(amount_label.release());
   }
 
   return container;
 }
 
 }  // namespace payments