Chromium Code Reviews| Index: chrome/browser/ui/views/autofill/autofill_popup_view_views.cc |
| diff --git a/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc b/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc |
| index 0e47429d67abc81e89cff00bcb1e6cac72cf32f0..063dbd69b4281a86a1e269b2becdfde0a50a596b 100644 |
| --- a/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc |
| +++ b/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc |
| @@ -70,8 +70,12 @@ void AutofillPopupViewViews::InvalidateRow(size_t row) { |
| void AutofillPopupViewViews::DrawAutofillEntry(gfx::Canvas* canvas, |
| int index, |
| const gfx::Rect& entry_rect) { |
| - if (controller_->selected_line() == index) |
| + if (controller_->selected_line() == index) { |
| canvas->FillRect(entry_rect, kHoveredBackgroundColor); |
| + } else if (controller_->GetSuggestionAt(index).frontend_id == |
| + POPUP_ITEM_ID_CREDIT_CARD_SIGNIN_PROMO) { |
| + canvas->FillRect(entry_rect, kPromoPopupBackground); |
|
Evan Stade
2016/07/15 17:27:08
can you move this color logic to the controller?
Mathieu
2016/07/15 19:30:20
Done.
Evan Stade
2016/07/15 19:33:38
it seems to still be here. Also I had intended for
Mathieu
2016/07/18 13:48:02
Sorry, moved the logic now.
|
| + } |
| const bool is_rtl = controller_->IsRTL(); |
| const int text_align = |