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

Unified Diff: chrome/browser/ui/views/extensions/media_galleries_dialog_views.cc

Issue 145033006: views: Make View::set_border() take a scoped_ptr<>. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase to ToT Created 6 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/extensions/media_galleries_dialog_views.cc
diff --git a/chrome/browser/ui/views/extensions/media_galleries_dialog_views.cc b/chrome/browser/ui/views/extensions/media_galleries_dialog_views.cc
index 7da680199b2913d2d17948e9a336690f63bd352f..765c32c779ab4705af7897348e067d552b6d9313 100644
--- a/chrome/browser/ui/views/extensions/media_galleries_dialog_views.cc
+++ b/chrome/browser/ui/views/extensions/media_galleries_dialog_views.cc
@@ -128,11 +128,11 @@ void MediaGalleriesDialogViews::InitChildViews() {
scroll_container->SetLayoutManager(new views::BoxLayout(
views::BoxLayout::kVertical, 0, 0,
views::kRelatedControlSmallVerticalSpacing));
- scroll_container->set_border(views::Border::CreateEmptyBorder(
- views::kRelatedControlVerticalSpacing,
- 0,
- views::kRelatedControlVerticalSpacing,
- 0));
+ scroll_container->SetBorder(
+ views::Border::CreateEmptyBorder(views::kRelatedControlVerticalSpacing,
+ 0,
+ views::kRelatedControlVerticalSpacing,
+ 0));
// Add attached galleries checkboxes.
checkbox_map_.clear();
@@ -161,11 +161,11 @@ void MediaGalleriesDialogViews::InitChildViews() {
controller_->GetUnattachedLocationsHeader());
unattached_text->SetMultiLine(true);
unattached_text->SetHorizontalAlignment(gfx::ALIGN_LEFT);
- unattached_text->set_border(views::Border::CreateEmptyBorder(
- views::kRelatedControlVerticalSpacing,
- views::kPanelHorizMargin,
- views::kRelatedControlVerticalSpacing,
- 0));
+ unattached_text->SetBorder(
+ views::Border::CreateEmptyBorder(views::kRelatedControlVerticalSpacing,
+ views::kPanelHorizMargin,
+ views::kRelatedControlVerticalSpacing,
+ 0));
scroll_container->AddChildView(unattached_text);
// Add unattached galleries checkboxes.
@@ -232,7 +232,7 @@ bool MediaGalleriesDialogViews::AddOrUpdateGallery(
secondary_text->SetTooltipText(tooltip_text);
secondary_text->SetEnabledColor(kDeemphasizedTextColor);
secondary_text->SetTooltipText(tooltip_text);
- secondary_text->set_border(views::Border::CreateEmptyBorder(
+ secondary_text->SetBorder(views::Border::CreateEmptyBorder(
0,
views::kRelatedControlSmallHorizontalSpacing,
0,
@@ -241,11 +241,8 @@ bool MediaGalleriesDialogViews::AddOrUpdateGallery(
views::View* checkbox_view = new views::View();
if (gallery.pref_id != kInvalidMediaGalleryPrefId)
checkbox_view->set_context_menu_controller(this);
- checkbox_view->set_border(views::Border::CreateEmptyBorder(
- 0,
- views::kPanelHorizMargin,
- trailing_vertical_space,
- 0));
+ checkbox_view->SetBorder(views::Border::CreateEmptyBorder(
+ 0, views::kPanelHorizMargin, trailing_vertical_space, 0));
checkbox_view->SetLayoutManager(
new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0, 0));
checkbox_view->AddChildView(checkbox);
« no previous file with comments | « chrome/browser/ui/views/extensions/extension_install_dialog_view.cc ('k') | chrome/browser/ui/views/find_bar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698