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

Unified Diff: chrome/browser/ui/views/layout_constants.cc

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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/views/layout_constants.h ('k') | chrome/browser/ui/views/tabs/tab.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/layout_constants.cc
diff --git a/chrome/browser/ui/views/layout_constants.cc b/chrome/browser/ui/views/layout_constants.cc
index 63924773fd82972277cc2cf58fee769db999b0cf..de59375c435eb208a224304b4f9e33a0ec334717 100644
--- a/chrome/browser/ui/views/layout_constants.cc
+++ b/chrome/browser/ui/views/layout_constants.cc
@@ -25,9 +25,11 @@ int GetLayoutConstant(LayoutConstant constant) {
const int kTabMaximumTitleWidth[] = {175, 175, 175};
const int kTabPinnedContentWidth[] = {25, 25, 25};
#if defined(OS_MACOSX)
+ const int kTabTopExclusionHeight[] = {0, 0, 0};
const int kTabstripNewTabButtonOverlap[] = {8, 8, 8};
const int kTabstripTabOverlap[] = {19, 19, 19};
#else
+ const int kTabTopExclusionHeight[] = {2, 2, 2};
const int kTabstripNewTabButtonOverlap[] = {11, 11, 11};
const int kTabstripTabOverlap[] = {26, 26, 26};
#endif
@@ -78,6 +80,8 @@ int GetLayoutConstant(LayoutConstant constant) {
return kTabMaximumTitleWidth[mode];
case TAB_PINNED_CONTENT_WIDTH:
return kTabPinnedContentWidth[mode];
+ case TAB_TOP_EXCLUSION_HEIGHT:
+ return kTabTopExclusionHeight[mode];
case TOOLBAR_CONTENT_SHADOW_HEIGHT:
return kToolbarContentShadowHeight[mode];
case TOOLBAR_CONTENT_SHADOW_HEIGHT_ASH:
« no previous file with comments | « chrome/browser/ui/views/layout_constants.h ('k') | chrome/browser/ui/views/tabs/tab.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698