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

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

Issue 1403413006: Distill the concept of the "tab top exclusion region", an area inside the tab (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@combine_background_image_getters
Patch Set: Fix indenting error Created 5 years, 1 month 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_VIEWS_LAYOUT_CONSTANTS_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_
6 #define CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_ 6 #define CHROME_BROWSER_UI_VIEWS_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 {
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 // The horizontal space between a tab's favicon and its title. 65 // The horizontal space between a tab's favicon and its title.
66 TAB_FAVICON_TITLE_SPACING, 66 TAB_FAVICON_TITLE_SPACING,
67 67
68 // The maximum width we'll allow for a tab's title, when the tabstrip is wide 68 // The maximum width we'll allow for a tab's title, when the tabstrip is wide
69 // enough for tabs to take as much space as they want. 69 // enough for tabs to take as much space as they want.
70 TAB_MAXIMUM_TITLE_WIDTH, 70 TAB_MAXIMUM_TITLE_WIDTH,
71 71
72 // Width available for content inside a pinned tab. 72 // Width available for content inside a pinned tab.
73 TAB_PINNED_CONTENT_WIDTH, 73 TAB_PINNED_CONTENT_WIDTH,
74 74
75 // Height of the region atop the tab bounds that is normally excluded when
76 // calculating the tab hit test region or considering where to paint a
77 // custom theme background. (Based on theme behavior it seems like at some
78 // point this was added to the area above the top of the tab images, and we
79 // now need to subtract it back off to compensate.)
80 TAB_TOP_EXCLUSION_HEIGHT,
81
75 // Non-ash uses a rounded content area with no shadow in the assets. 82 // Non-ash uses a rounded content area with no shadow in the assets.
76 // Ash doesn't use a rounded content area and its top edge has an extra 83 // Ash doesn't use a rounded content area and its top edge has an extra
77 // shadow. 84 // shadow.
78 TOOLBAR_CONTENT_SHADOW_HEIGHT, 85 TOOLBAR_CONTENT_SHADOW_HEIGHT,
79 TOOLBAR_CONTENT_SHADOW_HEIGHT_ASH, 86 TOOLBAR_CONTENT_SHADOW_HEIGHT_ASH,
80 87
81 // Additional horizontal padding between the elements in the toolbar. 88 // Additional horizontal padding between the elements in the toolbar.
82 TOOLBAR_ELEMENT_PADDING, 89 TOOLBAR_ELEMENT_PADDING,
83 90
84 // Padding between the right-edge of the location bar and browser actions. 91 // Padding between the right-edge of the location bar and browser actions.
(...skipping 28 matching lines...) Expand all
113 TOOLBAR, 120 TOOLBAR,
114 121
115 // The spacing between a ToolbarButton's image and its border. 122 // The spacing between a ToolbarButton's image and its border.
116 TOOLBAR_BUTTON, 123 TOOLBAR_BUTTON,
117 }; 124 };
118 125
119 int GetLayoutConstant(LayoutConstant constant); 126 int GetLayoutConstant(LayoutConstant constant);
120 gfx::Insets GetLayoutInsets(LayoutInset inset); 127 gfx::Insets GetLayoutInsets(LayoutInset inset);
121 128
122 #endif // CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_ 129 #endif // CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/frame/opaque_browser_frame_view_layout.cc ('k') | chrome/browser/ui/views/layout_constants.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698