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

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: Further renaming 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..2359aa41fb2bee5b645587b8d32f0df1ec14f0eb 100644
--- a/chrome/browser/ui/views/extensions/media_galleries_dialog_views.cc
+++ b/chrome/browser/ui/views/extensions/media_galleries_dialog_views.cc
@@ -128,7 +128,7 @@ void MediaGalleriesDialogViews::InitChildViews() {
scroll_container->SetLayoutManager(new views::BoxLayout(
views::BoxLayout::kVertical, 0, 0,
views::kRelatedControlSmallVerticalSpacing));
- scroll_container->set_border(views::Border::CreateEmptyBorder(
+ scroll_container->SetBorder(views::Border::CreateEmptyBorder(
views::kRelatedControlVerticalSpacing,
0,
views::kRelatedControlVerticalSpacing,
@@ -161,7 +161,7 @@ void MediaGalleriesDialogViews::InitChildViews() {
controller_->GetUnattachedLocationsHeader());
unattached_text->SetMultiLine(true);
unattached_text->SetHorizontalAlignment(gfx::ALIGN_LEFT);
- unattached_text->set_border(views::Border::CreateEmptyBorder(
+ unattached_text->SetBorder(views::Border::CreateEmptyBorder(
views::kRelatedControlVerticalSpacing,
views::kPanelHorizMargin,
views::kRelatedControlVerticalSpacing,
@@ -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,7 +241,7 @@ 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(
+ checkbox_view->SetBorder(views::Border::CreateEmptyBorder(
0,
views::kPanelHorizMargin,
trailing_vertical_space,

Powered by Google App Engine
This is Rietveld 408576698