[Autofill] Tweak credit card signin promo visually.

chrome/browser/ui/views/autofill/autofill_popup_view_views.cc

 // Copyright (c) 2012 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/autofill/autofill_popup_view_views.h"
 
 #include "chrome/browser/ui/autofill/autofill_popup_controller.h"
 #include "chrome/browser/ui/autofill/autofill_popup_layout_model.h"
 #include "components/autofill/core/browser/popup_item_ids.h"
 #include "components/autofill/core/browser/suggestion.h"
@@ skipping 52 matching lines @@
   }
 }
 
 void AutofillPopupViewViews::InvalidateRow(size_t row) {
   SchedulePaintInRect(controller_->layout_model().GetRowBounds(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 the hovered bg color logic to
move as well.

[Mathieu, 2016/07/18 13:48:02]

Sorry, moved the logic now.

+  }
 
   const bool is_rtl = controller_->IsRTL();
   const int text_align =
       is_rtl ? gfx::Canvas::TEXT_ALIGN_RIGHT : gfx::Canvas::TEXT_ALIGN_LEFT;
   gfx::Rect value_rect = entry_rect;
   value_rect.Inset(AutofillPopupLayoutModel::kEndPadding, 0);
   canvas->DrawStringRectWithFlags(
       controller_->GetElidedValueAt(index),
       controller_->GetValueFontListForRow(index),
       controller_->IsWarning(index) ? kWarningTextColor : kValueTextColor,
@@ skipping 45 matching lines @@
 
   // If the top level widget can't be found, cancel the popup since we can't
   // fully set it up.
   if (!observing_widget)
     return NULL;
 
   return new AutofillPopupViewViews(controller, observing_widget);
 }
 
 }  // namespace autofill