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

Side by Side Diff: chrome/browser/ui/autofill/autofill_popup_view.h

Issue 14053015: Don't add subtext padding if there is no subpadding. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_
6 #define CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_ 6 #define CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_
7 7
8 #include "ui/gfx/native_widget_types.h" 8 #include "ui/gfx/native_widget_types.h"
9 9
10 namespace gfx { 10 namespace gfx {
11 class Rect; 11 class Rect;
12 } 12 }
13 13
14 namespace ui { 14 namespace ui {
15 class KeyEvent; 15 class KeyEvent;
16 } 16 }
17 17
18 namespace autofill { 18 namespace autofill {
19 19
20 class AutofillPopupController; 20 class AutofillPopupController;
21 21
22 // The interface for creating and controlling a platform-dependent 22 // The interface for creating and controlling a platform-dependent
23 // AutofillPopupView. 23 // AutofillPopupView.
24 class AutofillPopupView { 24 class AutofillPopupView {
25 public: 25 public:
26 // The size of the border around the entire results popup, in pixels. 26 // The size of the border around the entire results popup, in pixels.
27 static const int kBorderThickness = 1; 27 static const int kBorderThickness = 1;
28 28
29 // The minimum amount of padding between the Autofill name and subtext,
30 // in pixels.
31 static const size_t kNamePadding = 15;
32
29 // The amount of padding between icons in pixels. 33 // The amount of padding between icons in pixels.
30 static const int kIconPadding = 5; 34 static const int kIconPadding = 5;
31 35
32 // The amount of padding at the end of the popup in pixels. 36 // The amount of padding at the end of the popup in pixels.
33 static const int kEndPadding = 3; 37 static const int kEndPadding = 3;
34 38
35 // Height of the delete icon in pixels. 39 // Height of the delete icon in pixels.
36 static const int kDeleteIconHeight = 16; 40 static const int kDeleteIconHeight = 16;
37 41
38 // Width of the delete icon in pixels. 42 // Width of the delete icon in pixels.
(...skipping 29 matching lines...) Expand all
68 private: 72 private:
69 // Used to check that the hide function was called, to check that the class 73 // Used to check that the hide function was called, to check that the class
70 // is only destroyed through the Hide function. Remove after Dev channel 74 // is only destroyed through the Hide function. Remove after Dev channel
71 // release. 75 // release.
72 bool hide_called_; 76 bool hide_called_;
73 }; 77 };
74 78
75 } // namespace autofill 79 } // namespace autofill
76 80
77 #endif // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_ 81 #endif // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698