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

Side by Side Diff: chrome/browser/ui/layout_constants.h

Issue 1505223004: Do not use assets for Ash window control button backgrounds (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: WIP for pkasting and estade feedback 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_LAYOUT_CONSTANTS_H_ 5 #ifndef CHROME_BROWSER_UI_LAYOUT_CONSTANTS_H_
6 #define CHROME_BROWSER_UI_LAYOUT_CONSTANTS_H_ 6 #define CHROME_BROWSER_UI_LAYOUT_CONSTANTS_H_
7 7
8 #include "ui/gfx/geometry/insets.h" 8 #include "ui/gfx/geometry/insets.h"
9 9
10 enum LayoutConstant { 10 enum LayoutConstant {
11 // The height of a button in in a maximized Ash browser window.
12 ASH_BROWSER_MAXIMIZED_HEADER_BUTTON_HEIGHT,
tdanderson 2015/12/17 18:00:54 Peter, is it appropriate to put these here since t
Peter Kasting 2015/12/17 20:56:08 Covered in my other reply.
tdanderson 2015/12/18 17:42:53 Acknowledged.
13
14 // The width of a button in in a maximized Ash browser window.
15 ASH_BROWSER_MAXIMIZED_HEADER_BUTTON_WIDTH,
16
17 // The height of a button in in a restored Ash browser window.
18 ASH_BROWSER_RESTORED_HEADER_BUTTON_HEIGHT,
19
20 // The width of a button in in a restored Ash browser window.
21 ASH_BROWSER_RESTORED_HEADER_BUTTON_WIDTH,
22
11 // Vertical offset from top of content to the top of find bar. 23 // Vertical offset from top of content to the top of find bar.
12 FIND_BAR_TOOLBAR_OVERLAP, 24 FIND_BAR_TOOLBAR_OVERLAP,
13 25
14 // Horizontal padding applied between items of icon-label views. 26 // Horizontal padding applied between items of icon-label views.
15 ICON_LABEL_VIEW_INTERNAL_PADDING, 27 ICON_LABEL_VIEW_INTERNAL_PADDING,
16 28
17 // Additional horizontal padding applied on the trailing edge of icon-label 29 // Additional horizontal padding applied on the trailing edge of icon-label
18 // views. 30 // views.
19 ICON_LABEL_VIEW_TRAILING_PADDING, 31 ICON_LABEL_VIEW_TRAILING_PADDING,
20 32
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 TOOLBAR, 133 TOOLBAR,
122 134
123 // The spacing between a ToolbarButton's image and its border. 135 // The spacing between a ToolbarButton's image and its border.
124 TOOLBAR_BUTTON, 136 TOOLBAR_BUTTON,
125 }; 137 };
126 138
127 int GetLayoutConstant(LayoutConstant constant); 139 int GetLayoutConstant(LayoutConstant constant);
128 gfx::Insets GetLayoutInsets(LayoutInset inset); 140 gfx::Insets GetLayoutInsets(LayoutInset inset);
129 141
130 #endif // CHROME_BROWSER_UI_LAYOUT_CONSTANTS_H_ 142 #endif // CHROME_BROWSER_UI_LAYOUT_CONSTANTS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698