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

Unified Diff: chrome/browser/ui/autofill/autofill_popup_layout_model.cc

Issue 2581513002: Replace hard-coded colors from ui::NativeTheme for consistency and accessibility. (Closed)
Patch Set: Resolved conflicts during patching Created 3 years, 11 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/autofill/autofill_popup_layout_model.cc
diff --git a/chrome/browser/ui/autofill/autofill_popup_layout_model.cc b/chrome/browser/ui/autofill/autofill_popup_layout_model.cc
index 6746c1655d75d1bda47058e7b3bd6e3dc3b39a6a..a481c95c2d0f6a129a48c37d7e2b4500209fa565 100644
--- a/chrome/browser/ui/autofill/autofill_popup_layout_model.cc
+++ b/chrome/browser/ui/autofill/autofill_popup_layout_model.cc
@@ -191,15 +191,16 @@ const gfx::FontList& AutofillPopupLayoutModel::GetLabelFontListForRow(
return smaller_font_list_;
}
-SkColor AutofillPopupLayoutModel::GetValueFontColorForRow(size_t index) const {
+ui::NativeTheme::ColorId AutofillPopupLayoutModel::GetValueFontColorIDForRow(
+ size_t index) const {
std::vector<autofill::Suggestion> suggestions = delegate_->GetSuggestions();
switch (suggestions[index].frontend_id) {
case POPUP_ITEM_ID_HTTP_NOT_SECURE_WARNING_MESSAGE:
- return gfx::kGoogleRed700;
+ return ui::NativeTheme::kColorId_AlertSeverityHigh;
case POPUP_ITEM_ID_INSECURE_CONTEXT_PAYMENT_DISABLED_MESSAGE:
- return kLabelTextColor;
+ return ui::NativeTheme::kColorId_ResultsTableNormalDimmedText;
default:
- return kValueTextColor;
+ return ui::NativeTheme::kColorId_ResultsTableNormalText;
}
}
« no previous file with comments | « chrome/browser/ui/autofill/autofill_popup_layout_model.h ('k') | chrome/browser/ui/autofill/popup_constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698