Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(132)

Unified Diff: chrome/browser/ui/autofill/autofill_popup_layout_model.h

Issue 2517843002: Http Bad: Put icon on the left of http warning message on Views (Closed)
Patch Set: fix mac Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..b3a2c735fbaf8a0531c8465a949cd677d6608700 100644
--- a/chrome/browser/ui/autofill/autofill_popup_layout_model.h
+++ b/chrome/browser/ui/autofill/autofill_popup_layout_model.h
@@ -32,9 +32,17 @@ 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 horizontal padding around icons in pixels for HTTP warning
+ // message.
+ static const int kHttpWarningIconPadding = 8;
+
// The amount of padding at the end of the popup in pixels.
static const int kEndPadding = 8;
@@ -62,7 +70,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;

Powered by Google App Engine
This is Rietveld 408576698