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

Unified Diff: chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc

Issue 2663013003: Rename various LayoutDelegate types/functions for brevity and consistency. (Closed)
Patch Set: Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc
diff --git a/chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc b/chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc
index 30ca1bf1e3eb68c4d6faac45e301cf51ef69689e..662e6d50eb97795e601da10cbf5ed8e2a728a764 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc
+++ b/chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc
@@ -69,11 +69,10 @@ views::View* ToolbarActionsBarBubbleViews::CreateExtraView() {
if (icon && label) {
views::View* parent = new views::View();
- parent->SetLayoutManager(
- new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0,
- LayoutDelegate::Get()->GetLayoutDistance(
- LayoutDelegate::LayoutDistanceType::
- RELATED_CONTROL_VERTICAL_SPACING)));
+ parent->SetLayoutManager(new views::BoxLayout(
+ views::BoxLayout::kHorizontal, 0, 0,
+ LayoutDelegate::Get()->GetMetric(
+ LayoutDelegate::Metric::RELATED_CONTROL_VERTICAL_SPACING)));
parent->AddChildView(icon.release());
parent->AddChildView(label.release());
return parent;
@@ -110,8 +109,8 @@ void ToolbarActionsBarBubbleViews::Init() {
LayoutDelegate* delegate = LayoutDelegate::Get();
SetLayoutManager(new views::BoxLayout(
views::BoxLayout::kVertical, 0, 0,
- delegate->GetLayoutDistance(LayoutDelegate::LayoutDistanceType::
- RELATED_CONTROL_VERTICAL_SPACING)));
+ delegate->GetMetric(
+ LayoutDelegate::Metric::RELATED_CONTROL_VERTICAL_SPACING)));
// Add the content string.
views::Label* content_label = new views::Label(
@@ -128,8 +127,9 @@ void ToolbarActionsBarBubbleViews::Init() {
if (!item_list.empty()) {
item_list_ = new views::Label(item_list);
item_list_->SetBorder(views::CreateEmptyBorder(
- 0, delegate->GetLayoutDistance(LayoutDelegate::LayoutDistanceType::
- RELATED_CONTROL_HORIZONTAL_SPACING),
+ 0,
+ delegate->GetMetric(
+ LayoutDelegate::Metric::RELATED_CONTROL_HORIZONTAL_SPACING),
0, 0));
item_list_->SetMultiLine(true);
item_list_->SizeToFit(width);

Powered by Google App Engine
This is Rietveld 408576698