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

Unified Diff: chrome/browser/ui/views/download/download_item_view.cc

Issue 2744463002: Add VectorIconButton functionality to ImageButton. (Closed)
Patch Set: WIP: use observer Created 3 years, 9 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/download/download_item_view.cc
diff --git a/chrome/browser/ui/views/download/download_item_view.cc b/chrome/browser/ui/views/download/download_item_view.cc
index fa2eff196f3d24b0b0a1132d49fcfb07c4d7476d..8a8291c90f9ec9005fb1e902118a4c1b50a70c8d 100644
--- a/chrome/browser/ui/views/download/download_item_view.cc
+++ b/chrome/browser/ui/views/download/download_item_view.cc
@@ -158,7 +158,9 @@ class SeparatorBorder : public views::FocusableBorder {
class DownloadItemView::DropDownButton : public views::VectorIconButton {
public:
explicit DropDownButton(views::VectorIconButtonDelegate* delegate)
- : views::VectorIconButton(delegate) {}
+ : views::VectorIconButton(delegate) {
+ SetPadding(gfx::Insets(kDropdownBorderWidth));
+ }
~DropDownButton() override {}
// Promoted visibility to public.
@@ -194,8 +196,6 @@ DownloadItemView::DownloadItemView(DownloadItem* download_item,
download()->AddObserver(this);
set_context_menu_controller(this);
- dropdown_button_->SetBorder(
- views::CreateEmptyBorder(gfx::Insets(kDropdownBorderWidth)));
AddChildView(dropdown_button_);
LoadIcon();

Powered by Google App Engine
This is Rietveld 408576698