Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6612)

Unified Diff: chrome/browser/ui/views/payments/payment_request_views_util.cc

Issue 2774233005: Creating PaymentOptionsProvider interface (Closed)
Patch Set: default case for compile Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/payments/payment_request_views_util.cc
diff --git a/chrome/browser/ui/views/payments/payment_request_views_util.cc b/chrome/browser/ui/views/payments/payment_request_views_util.cc
index 17b4c6aed1eaededd41b0a3b147ffe97bf0a4c9e..361b2904bc99e290818f410adbf419c9d7583a1e 100644
--- a/chrome/browser/ui/views/payments/payment_request_views_util.cc
+++ b/chrome/browser/ui/views/payments/payment_request_views_util.cc
@@ -207,23 +207,21 @@ std::unique_ptr<views::View> GetContactInfoLabel(
AddressStyleType type,
const std::string& locale,
const autofill::AutofillProfile& profile,
- bool show_payer_name,
- bool show_payer_phone,
- bool show_payer_email) {
+ PaymentOptionsProvider* options) {
base::string16 name =
- show_payer_name
+ options->request_payer_name()
? profile.GetInfo(autofill::AutofillType(autofill::NAME_FULL), locale)
: base::string16();
base::string16 phone =
- show_payer_phone
+ options->request_payer_phone()
? profile.GetInfo(
autofill::AutofillType(autofill::PHONE_HOME_WHOLE_NUMBER),
locale)
: base::string16();
base::string16 email =
- show_payer_email
+ options->request_payer_email()
? profile.GetInfo(autofill::AutofillType(autofill::EMAIL_ADDRESS),
locale)
: base::string16();
@@ -247,13 +245,13 @@ std::unique_ptr<views::Label> CreateBoldLabel(const base::string16& text) {
}
base::string16 GetShippingAddressSectionString(
- payments::mojom::PaymentShippingType shipping_type) {
+ PaymentShippingType shipping_type) {
switch (shipping_type) {
- case payments::mojom::PaymentShippingType::DELIVERY:
+ case PaymentShippingType::DELIVERY:
return l10n_util::GetStringUTF16(IDS_PAYMENTS_DELIVERY_ADDRESS_LABEL);
- case payments::mojom::PaymentShippingType::PICKUP:
+ case PaymentShippingType::PICKUP:
return l10n_util::GetStringUTF16(IDS_PAYMENTS_PICKUP_ADDRESS_LABEL);
- case payments::mojom::PaymentShippingType::SHIPPING:
+ case PaymentShippingType::SHIPPING:
return l10n_util::GetStringUTF16(IDS_PAYMENTS_SHIPPING_ADDRESS_LABEL);
}
// MSVC doesn't compile with only the above switch statement because it can't
@@ -262,13 +260,13 @@ base::string16 GetShippingAddressSectionString(
}
base::string16 GetShippingOptionSectionString(
- payments::mojom::PaymentShippingType shipping_type) {
+ PaymentShippingType shipping_type) {
switch (shipping_type) {
- case payments::mojom::PaymentShippingType::DELIVERY:
+ case PaymentShippingType::DELIVERY:
return l10n_util::GetStringUTF16(IDS_PAYMENTS_DELIVERY_OPTION_LABEL);
- case payments::mojom::PaymentShippingType::PICKUP:
+ case PaymentShippingType::PICKUP:
return l10n_util::GetStringUTF16(IDS_PAYMENTS_PICKUP_OPTION_LABEL);
- case payments::mojom::PaymentShippingType::SHIPPING:
+ case PaymentShippingType::SHIPPING:
return l10n_util::GetStringUTF16(IDS_PAYMENTS_SHIPPING_OPTION_LABEL);
}
// MSVC doesn't compile with only the above switch statement because it can't

Powered by Google App Engine
This is Rietveld 408576698