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

Unified Diff: chrome/browser/ui/views/frame/opaque_browser_frame_view_layout.cc

Issue 2555623002: Last round of cleaning up MD layout constants. (Closed)
Patch Set: pkasting review, remove debug code, rebase Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/frame/opaque_browser_frame_view_layout.cc
diff --git a/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout.cc b/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout.cc
index 36ed8cc08c272fe1e76b9f185a4963b8b033fb19..398fcf0971aacdac0cbfa4a58c951c8cb8b9a125 100644
--- a/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout.cc
+++ b/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout.cc
@@ -107,7 +107,7 @@ void OpaqueBrowserFrameViewLayout::SetButtonOrdering(
gfx::Rect OpaqueBrowserFrameViewLayout::GetBoundsForTabStrip(
const gfx::Size& tabstrip_preferred_size,
int available_width) const {
- int x = leading_button_start_ + GetLayoutConstant(AVATAR_ICON_PADDING);
+ int x = leading_button_start_ + OpaqueBrowserFrameView::kAvatarIconPadding;
available_width -= x + NewTabCaptionSpacing() + trailing_button_start_;
return gfx::Rect(x, GetTabStripInsetsTop(false), std::max(0, available_width),
tabstrip_preferred_size.height());
@@ -369,7 +369,7 @@ void OpaqueBrowserFrameViewLayout::LayoutIncognitoIcon(views::View* host) {
trailing_button_start_ = std::max(trailing_button_start_, min_button_width);
if (incognito_icon_) {
- const int pad = GetLayoutConstant(AVATAR_ICON_PADDING);
+ constexpr int pad = OpaqueBrowserFrameView::kAvatarIconPadding;
const gfx::Size size(delegate_->GetIncognitoAvatarIcon().size());
const int incognito_width = pad + size.width();
int x;

Powered by Google App Engine
This is Rietveld 408576698