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

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

Issue 2971783002: Skeleton for showing "Show all saved passwords row" for Linux/CrOs/Windows platforms (Closed)
Patch Set: . Created 3 years, 5 months 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 f82711148f4150ead6c589d3d6926def93f9b9c5..14e533f6708d5ab353b7d38a4c7659c7a08b89e3 100644
--- a/chrome/browser/ui/autofill/autofill_popup_layout_model.h
+++ b/chrome/browser/ui/autofill/autofill_popup_layout_model.h
@@ -32,21 +32,21 @@ class AutofillPopupLayoutModel {
~AutofillPopupLayoutModel();
// The minimum amount of padding between the Autofill name and subtext,
- // in pixels.
+ // in dpi.
vasilii 2017/07/13 16:11:51 I think Evan meant Device-independent pixel, not D
Evan Stade 2017/07/13 16:48:43 that's right.
melandory 2017/07/25 15:08:50 Done.
static const int kNamePadding = 15;
// The minimum amount of padding between the Autofill http warning message
- // name and subtext, in pixels.
+ // name and subtext, in dpi.
static const int kHttpWarningNamePadding = 8;
- // The amount of padding around icons in pixels.
+ // The amount of padding around icons in dpi.
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 horizontal padding around icons in dpi for the case, when
+ // icon is located on the left side.
+ static const int kPaddingBetweenLeftSideIconAndText = 8;
- // The amount of padding at the end of the popup in pixels.
+ // The amount of padding at the end of the popup in dpi.
static const int kEndPadding = 8;
#if !defined(OS_ANDROID)

Powered by Google App Engine
This is Rietveld 408576698