Http Bad: Add a PopupItemId to identify http warning message

chrome/browser/ui/autofill/autofill_popup_layout_model.cc

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 652017dcad23f5abd8fcc87a074a9c2d05977094..5eedd420f09260b5aade131fc80507da32d98576 100644
--- a/chrome/browser/ui/autofill/autofill_popup_layout_model.cc
+++ b/chrome/browser/ui/autofill/autofill_popup_layout_model.cc
@@ -157,8 +157,9 @@ const gfx::FontList& AutofillPopupLayoutModel::GetValueFontListForRow(
   // All other message types are defined here.
   PopupItemId id = static_cast<PopupItemId>(suggestions[index].frontend_id);
   switch (id) {
-    case POPUP_ITEM_ID_WARNING_MESSAGE:
+    case POPUP_ITEM_ID_INSECURE_CONTEXT_PAYMENT_DISABLED_MESSAGE:
       return warning_font_list_;
+    case POPUP_ITEM_ID_HTTP_NOT_SECURE_WARNING_MESSAGE:
     case POPUP_ITEM_ID_CLEAR_FORM:
     case POPUP_ITEM_ID_CREDIT_CARD_SIGNIN_PROMO:
     case POPUP_ITEM_ID_AUTOFILL_OPTIONS:
@@ -178,6 +179,19 @@ const gfx::FontList& AutofillPopupLayoutModel::GetValueFontListForRow(
 const gfx::FontList& AutofillPopupLayoutModel::GetLabelFontList() const {
   return smaller_font_list_;
 }
+
+SkColor AutofillPopupLayoutModel::GetValueFontColorForRow(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 kHttpWarningTextColor;
+    case POPUP_ITEM_ID_INSECURE_CONTEXT_PAYMENT_DISABLED_MESSAGE:
+      return kLabelTextColor;
+    default:
+      return kValueTextColor;
+  }
+}
+

[Mathieu, 2016/11/14 14:40:15]

nit: remove extra line

[lshang, 2016/11/15 04:36:40]

Done.

 #endif
 
 int AutofillPopupLayoutModel::LineFromY(int y) const {