Index: chrome/browser/ui/views/payments/payment_sheet_view_controller.cc |
diff --git a/chrome/browser/ui/views/payments/payment_sheet_view_controller.cc b/chrome/browser/ui/views/payments/payment_sheet_view_controller.cc |
index 704d942a44092068228c8dc6b015ed8640f57496..8d683122725aebb44ef6a5643e48cafc36685e14 100644 |
--- a/chrome/browser/ui/views/payments/payment_sheet_view_controller.cc |
+++ b/chrome/browser/ui/views/payments/payment_sheet_view_controller.cc |
@@ -20,6 +20,7 @@ |
#include "chrome/browser/browser_process.h" |
#include "chrome/browser/profiles/profile.h" |
#include "chrome/browser/ui/chrome_pages.h" |
+#include "chrome/browser/ui/views/harmony/chrome_typography.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" |
@@ -84,8 +85,9 @@ class PreviewEliderLabel : public views::Label { |
// and |n| is the "N more" item count. |
PreviewEliderLabel(const base::string16& preview_text, |
const base::string16& format_string, |
- int n) |
- : views::Label(base::string16()), |
+ int n, |
+ int text_style) |
+ : views::Label(base::string16(), views::style::CONTEXT_LABEL, text_style), |
preview_text_(preview_text), |
format_string_(format_string), |
n_(n) {} |
@@ -193,7 +195,7 @@ std::unique_ptr<views::Button> CreatePaymentSheetRow( |
std::unique_ptr<views::View> CreateInlineCurrencyAmountItem( |
const base::string16& currency, |
const base::string16& amount, |
- bool disabled_color, |
+ bool hint_color, |
bool bold) { |
std::unique_ptr<views::View> item_amount_line = |
base::MakeUnique<views::View>(); |
@@ -207,15 +209,15 @@ std::unique_ptr<views::View> CreateInlineCurrencyAmountItem( |
views::GridLayout::LEADING, 1, |
views::GridLayout::USE_PREF, 0, 0); |
- std::unique_ptr<views::Label> currency_label = |
- bold ? CreateBoldLabel(currency) |
- : base::MakeUnique<views::Label>(currency); |
- if (disabled_color) { |
- currency_label->SetDisabledColor( |
- currency_label->GetNativeTheme()->GetSystemColor( |
- ui::NativeTheme::kColorId_LabelDisabledColor)); |
- currency_label->SetEnabled(false); |
- } |
+ DCHECK(!bold || !hint_color); |
+ std::unique_ptr<views::Label> currency_label; |
+ if (bold) |
+ currency_label = CreateBoldLabel(currency); |
+ else if (hint_color) |
+ currency_label = CreateHintLabel(currency); |
+ else |
+ currency_label = base::MakeUnique<views::Label>(currency); |
+ |
std::unique_ptr<views::Label> amount_label = |
bold ? CreateBoldLabel(amount) : base::MakeUnique<views::Label>(amount); |
amount_label->SetMultiLine(true); |
@@ -287,15 +289,8 @@ class PaymentSheetRowBuilder { |
const base::string16& truncated_content, |
const base::string16& button_string, |
bool button_enabled) { |
- std::unique_ptr<views::Label> content_view = |
- base::MakeUnique<views::Label>(truncated_content); |
- content_view->SetHorizontalAlignment(gfx::ALIGN_LEFT); |
- content_view->SetDisabledColor( |
- content_view->GetNativeTheme()->GetSystemColor( |
- ui::NativeTheme::kColorId_LabelDisabledColor)); |
- content_view->SetEnabled(false); |
- return CreateWithButton(std::move(content_view), button_string, |
- button_enabled); |
+ return CreateWithButton(CreateHintLabel(truncated_content, gfx::ALIGN_LEFT), |
+ button_string, button_enabled); |
} |
// Creates a row with a button in place of the chevron with the string between |
@@ -312,12 +307,9 @@ class PaymentSheetRowBuilder { |
const base::string16& button_string, |
bool button_enabled) { |
std::unique_ptr<PreviewEliderLabel> content_view = |
- base::MakeUnique<PreviewEliderLabel>(preview_text, format_string, n); |
+ base::MakeUnique<PreviewEliderLabel>(preview_text, format_string, n, |
+ STYLE_HINT); |
content_view->SetHorizontalAlignment(gfx::ALIGN_LEFT); |
- content_view->SetDisabledColor( |
- content_view->GetNativeTheme()->GetSystemColor( |
- ui::NativeTheme::kColorId_LabelDisabledColor)); |
- content_view->SetEnabled(false); |
return CreateWithButton(std::move(content_view), button_string, |
button_enabled); |
} |
@@ -554,20 +546,13 @@ PaymentSheetViewController::CreatePaymentSheetSummaryRow() { |
if (hidden_item_count > 0) { |
layout->StartRow(0, 0); |
std::unique_ptr<views::Label> label = |
- base::MakeUnique<views::Label>(l10n_util::GetPluralStringFUTF16( |
+ CreateHintLabel(l10n_util::GetPluralStringFUTF16( |
IDS_PAYMENT_REQUEST_ORDER_SUMMARY_MORE_ITEMS, hidden_item_count)); |
- label->SetDisabledColor(label->GetNativeTheme()->GetSystemColor( |
- ui::NativeTheme::kColorId_LabelDisabledColor)); |
- label->SetEnabled(false); |
layout->AddView(label.release()); |
if (is_mixed_currency) { |
std::unique_ptr<views::Label> multiple_currency_label = |
- base::MakeUnique<views::Label>(l10n_util::GetStringUTF16( |
+ CreateHintLabel(l10n_util::GetStringUTF16( |
IDS_PAYMENT_REQUEST_ORDER_SUMMARY_MULTIPLE_CURRENCY_INDICATOR)); |
- multiple_currency_label->SetDisabledColor( |
- multiple_currency_label->GetNativeTheme()->GetSystemColor( |
- ui::NativeTheme::kColorId_LabelDisabledColor)); |
- multiple_currency_label->SetEnabled(false); |
layout->AddView(multiple_currency_label.release()); |
} |
} |