Chromium Code Reviews| Index: chrome/browser/ui/views/frame/glass_browser_frame_view.cc |
| diff --git a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc |
| index 0ceb12d4e1b25e047ee80d78763a710386f44015..84a81d7696ab03bc022737e1c2e1f50ad7e05da6 100644 |
| --- a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc |
| +++ b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc |
| @@ -139,8 +139,7 @@ GlassBrowserFrameView::~GlassBrowserFrameView() { |
| gfx::Rect GlassBrowserFrameView::GetBoundsForTabStrip( |
| views::View* tabstrip) const { |
| - const int x = |
| - incognito_bounds_.right() + GetLayoutConstant(AVATAR_ICON_PADDING); |
| + const int x = incognito_bounds_.right() + kAvatarIconPadding; |
| int end_x = width() - ClientBorderThickness(false); |
| if (!CaptionButtonsOnLeadingEdge()) { |
| end_x = std::min(MinimizeButtonX(), end_x) - |
| @@ -739,7 +738,7 @@ void GlassBrowserFrameView::LayoutProfileSwitcher() { |
| } |
| void GlassBrowserFrameView::LayoutIncognitoIcon() { |
| - const int pad = GetLayoutConstant(AVATAR_ICON_PADDING); |
| + const int pad = kAvatarIconPadding; |
|
Peter Kasting
2016/12/08 20:57:49
Nit: Use directly below? Or make constexpr
Evan Stade
2016/12/13 01:31:17
Done.
|
| const gfx::Size size(GetIncognitoAvatarIcon().size()); |
| int x = ClientBorderThickness(false); |
| // In RTL, the icon needs to start after the caption buttons. |