Http Bad: Add a PopupItemId to identify http warning message

chrome/browser/ui/autofill/autofill_popup_controller.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_CONTROLLER_H_
 #define CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_CONTROLLER_H_
 
 #include <stddef.h>
 
 #include <vector>
@@ skipping 41 matching lines @@
                                           base::string16* title,
                                           base::string16* body) = 0;
 
   // Removes the suggestion at the given index.
   virtual bool RemoveSuggestion(int index) = 0;
 
   // Returns the background color of the row item according to its |index|, or
   // 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;
 
   virtual const AutofillPopupLayoutModel& layout_model() const = 0;
 
  protected:
   ~AutofillPopupController() override {}
 };
 
 }  // namespace autofill
 
 #endif  // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_CONTROLLER_H_