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

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

Issue 1398073002: Fix a variety of tabstrip positioning issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@parametrize
Patch Set: Remove erroneous comment Created 5 years, 2 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
« no previous file with comments | « chrome/browser/ui/views/frame/glass_browser_frame_view.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #include "chrome/browser/ui/views/layout_constants.h" 5 #include "chrome/browser/ui/views/layout_constants.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "ui/base/resource/material_design/material_design_controller.h" 8 #include "ui/base/resource/material_design/material_design_controller.h"
9 9
10 int GetLayoutConstant(LayoutConstant constant) { 10 int GetLayoutConstant(LayoutConstant constant) {
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 return kToolbarLocationBarRightPadding[mode]; 82 return kToolbarLocationBarRightPadding[mode];
83 case TOOLBAR_STANDARD_SPACING: 83 case TOOLBAR_STANDARD_SPACING:
84 return kToolbarStandardSpacing[mode]; 84 return kToolbarStandardSpacing[mode];
85 } 85 }
86 NOTREACHED(); 86 NOTREACHED();
87 return 0; 87 return 0;
88 } 88 }
89 89
90 gfx::Insets GetLayoutInsets(LayoutInset inset) { 90 gfx::Insets GetLayoutInsets(LayoutInset inset) {
91 const int kAvatarLeftPadding[] = {2, 4, 4}; 91 const int kAvatarLeftPadding[] = {2, 4, 4};
92 const int kAvatarRightPadding[] = {2, 2, 2}; 92 const int kAvatarRightPadding[] = {-6, 4, 4};
93 const int kAvatarBottomPadding[] = {2, 4, 4}; 93 const int kAvatarBottomPadding[] = {2, 4, 4};
94 const int kOmniboxDropdownIconPadding[] = {2, 4, 8}; 94 const int kOmniboxDropdownIconPadding[] = {2, 4, 8};
95 const int kOmniboxDropdownTextPadding[] = {3, 4, 8}; 95 const int kOmniboxDropdownTextPadding[] = {3, 4, 8};
96 const int kTabBottomPadding[] = {2, 2, 2}; 96 const int kTabBottomPadding[] = {2, 2, 2};
97 const int kTabLeftPadding[] = {20, 20, 20}; 97 const int kTabLeftPadding[] = {20, 20, 20};
98 const int kTabRightPadding[] = {20, 20, 20}; 98 const int kTabRightPadding[] = {20, 20, 20};
99 const int kTabTopPadding[] = {4, 4, 4}; 99 const int kTabTopPadding[] = {4, 4, 4};
100 const int kToolbarBottomPadding[] = {5, 5, 5}; 100 const int kToolbarBottomPadding[] = {5, 5, 5};
101 const int kToolbarButtonPadding[] = {2, 6, 6}; 101 const int kToolbarButtonPadding[] = {2, 6, 6};
102 const int kToolbarLeftPadding[] = {3, 4, 8}; 102 const int kToolbarLeftPadding[] = {3, 4, 8};
(...skipping 22 matching lines...) Expand all
125 kToolbarBottomPadding[mode], 125 kToolbarBottomPadding[mode],
126 kToolbarRightPadding[mode]); 126 kToolbarRightPadding[mode]);
127 case TOOLBAR_BUTTON: { 127 case TOOLBAR_BUTTON: {
128 const int inset = kToolbarButtonPadding[mode]; 128 const int inset = kToolbarButtonPadding[mode];
129 return gfx::Insets(inset, inset, inset, inset); 129 return gfx::Insets(inset, inset, inset, inset);
130 } 130 }
131 } 131 }
132 NOTREACHED(); 132 NOTREACHED();
133 return gfx::Insets(); 133 return gfx::Insets();
134 } 134 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/frame/glass_browser_frame_view.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698