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

Unified Diff: chrome/browser/ui/views/tabs/tab.cc

Issue 2555623002: Last round of cleaning up MD layout constants. (Closed)
Patch Set: 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/tabs/tab.cc
diff --git a/chrome/browser/ui/views/tabs/tab.cc b/chrome/browser/ui/views/tabs/tab.cc
index 4e894358c45078d50ad84550318e0cc457fed2ce..24dd623d05285ee4317329de1e7f2d4f84be895e 100644
--- a/chrome/browser/ui/views/tabs/tab.cc
+++ b/chrome/browser/ui/views/tabs/tab.cc
@@ -739,8 +739,8 @@ int Tab::GetTouchWidth() {
// static
int Tab::GetPinnedWidth() {
- return GetMinimumInactiveSize().width() +
- GetLayoutConstant(TAB_PINNED_CONTENT_WIDTH);
+ const int kTabPinnedContentWidth = 23;
+ return GetMinimumInactiveSize().width() + kTabPinnedContentWidth;
}
// static
@@ -919,9 +919,9 @@ void Tab::Layout() {
// Size the title to fill the remaining width and use all available height.
const bool show_title = ShouldRenderAsNormalTab();
if (show_title) {
- const int title_spacing = GetLayoutConstant(TAB_FAVICON_TITLE_SPACING);
- int title_left = showing_icon_ ?
- (favicon_bounds_.right() + title_spacing) : start;
+ const int kTitleSpacing = 6;
+ int title_left =
+ showing_icon_ ? (favicon_bounds_.right() + kTitleSpacing) : start;
int title_width = lb.right() - title_left;
if (showing_alert_indicator_) {
title_width =

Powered by Google App Engine
This is Rietveld 408576698