Http Bad: Put icon on the left of http warning message on Views

chrome/browser/ui/autofill/autofill_popup_layout_model.h

Index: chrome/browser/ui/autofill/autofill_popup_layout_model.h
diff --git a/chrome/browser/ui/autofill/autofill_popup_layout_model.h b/chrome/browser/ui/autofill/autofill_popup_layout_model.h
index 560d5bc0eb13fa5336d9977ef627c97fc7cd7fb2..3cbfae282c243e59e5cb5b22b72ff02da4c2c953 100644
--- a/chrome/browser/ui/autofill/autofill_popup_layout_model.h
+++ b/chrome/browser/ui/autofill/autofill_popup_layout_model.h
@@ -32,9 +32,16 @@ class AutofillPopupLayoutModel {
   // in pixels.
   static const int kNamePadding = 15;
 
-  // The amount of padding between icons in pixels.
+  // The minimum amount of padding between the Autofill http warning message
+  // name and subtext, in pixels.
+  static const int kHttpWarningNamePadding = 8;
+
+  // The amount of padding around icons in pixels.
   static const int kIconPadding = 5;
 
+  // The amount of padding around icons in pixels for HTTP warning message.

[Evan Stade, 2016/11/30 16:22:30]

nit: add the word horizontal?

[lshang, 2016/12/01 07:42:14]

Done.

+  static const int kHttpWarningIconPadding = 8;
+
   // The amount of padding at the end of the popup in pixels.
   static const int kEndPadding = 8;
 
@@ -62,7 +69,7 @@ class AutofillPopupLayoutModel {
   // The same font can vary based on the type of data it is showing at the row
   // |index|.
   const gfx::FontList& GetValueFontListForRow(size_t index) const;
-  const gfx::FontList& GetLabelFontList() const;
+  const gfx::FontList& GetLabelFontListForRow(size_t index) const;
 
   // Returns the value font color of the row item according to its |index|.
   SkColor GetValueFontColorForRow(size_t index) const;