Http Bad: Add a PopupItemId to identify http warning message

chrome/browser/ui/autofill/autofill_popup_controller.h

Index: chrome/browser/ui/autofill/autofill_popup_controller.h
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller.h b/chrome/browser/ui/autofill/autofill_popup_controller.h
index 53a729245a1cde659304023a8969dd683404cf25..fe5d046efca3b9d65c5f5ee592cfbfba73970cf9 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller.h
+++ b/chrome/browser/ui/autofill/autofill_popup_controller.h
@@ -59,6 +59,9 @@ class AutofillPopupController : public AutofillPopupViewDelegate {
   // transparent if the default popup background should be used.
   virtual SkColor GetBackgroundColorForRow(int index) const = 0;
 
+  // Returns the text color of the row item according to its |index|.
+  virtual SkColor GetTextColorForRow(int index) const = 0;

[Mathieu, 2016/11/12 12:52:05]

would name GetValueFontColorForRow and ideally would go in the
AutofillPopupLayoutModel (also bring along GetBackgroundColorForRow if possible)

[lshang, 2016/11/14 10:51:01]

Renamed GetValueFontColorForRow and moved it into layout model, but
GetBackgroundColorForRow can't be moved because it depends on selected_line_ in
controller.

+
   // Returns the index of the selected line. A line is "selected" when it is
   // hovered or has keyboard focus.
   virtual int selected_line() const = 0;