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

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

Issue 1522563002: Modernize autofill popup design (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 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 {
(...skipping 14 matching lines...) Expand all
25 public: 25 public:
26 // The minimum amount of padding between the Autofill name and subtext, 26 // The minimum amount of padding between the Autofill name and subtext,
27 // in pixels. 27 // in pixels.
28 static const size_t kNamePadding = 15; 28 static const size_t kNamePadding = 15;
29 29
30 // The amount of padding between icons in pixels. 30 // The amount of padding between icons in pixels.
31 static const int kIconPadding = 5; 31 static const int kIconPadding = 5;
32 32
33 // The amount of padding at the end of the popup in pixels. 33 // The amount of padding at the end of the popup in pixels.
34 static const int kEndPadding = 3; 34 static const int kEndPadding = 3;
35 static const int kMaterialEndPadding = 18;
35 36
36 // Height of the delete icon in pixels. 37 // Height of the delete icon in pixels.
37 static const int kDeleteIconHeight = 16; 38 static const int kDeleteIconHeight = 16;
38 39
39 // Width of the delete icon in pixels. 40 // Width of the delete icon in pixels.
40 static const int kDeleteIconWidth = 16; 41 static const int kDeleteIconWidth = 16;
41 42
42 // Displays the Autofill popup and fills it in with data from the controller. 43 // Displays the Autofill popup and fills it in with data from the controller.
43 virtual void Show() = 0; 44 virtual void Show() = 0;
44 45
45 // Hides the popup from view. This will cause the popup to be deleted. 46 // Hides the popup from view. This will cause the popup to be deleted.
46 virtual void Hide() = 0; 47 virtual void Hide() = 0;
47 48
48 // Invalidates the given row and redraw it. 49 // Invalidates the given row and redraw it.
49 virtual void InvalidateRow(size_t row) = 0; 50 virtual void InvalidateRow(size_t row) = 0;
50 51
51 // Refreshes the position of the popup. 52 // Refreshes the position of the popup.
52 virtual void UpdateBoundsAndRedrawPopup() = 0; 53 virtual void UpdateBoundsAndRedrawPopup() = 0;
53 54
54 // Factory function for creating the view. 55 // Factory function for creating the view.
55 static AutofillPopupView* Create(AutofillPopupController* controller); 56 static AutofillPopupView* Create(AutofillPopupController* controller);
56 57
57 protected: 58 protected:
58 virtual ~AutofillPopupView() {} 59 virtual ~AutofillPopupView() {}
59 }; 60 };
60 61
61 } // namespace autofill 62 } // namespace autofill
62 63
63 #endif // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_ 64 #endif // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_POPUP_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698