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

Unified Diff: chrome/browser/ui/views/page_info/website_settings_popup_view.cc

Issue 2750483003: Harmony - Remove icon from internal page info dialog view in Material Design mode. (Closed)
Patch Set: Use the new ShouldShowWindowIcon() method on LayoutDelegate 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/page_info/website_settings_popup_view.cc
diff --git a/chrome/browser/ui/views/page_info/website_settings_popup_view.cc b/chrome/browser/ui/views/page_info/website_settings_popup_view.cc
index dbfc2513b8df7830327f3125f99ab7c1adc8c539..6b97cba697cd8199f4e046ba803de044e5043c72 100644
--- a/chrome/browser/ui/views/page_info/website_settings_popup_view.cc
+++ b/chrome/browser/ui/views/page_info/website_settings_popup_view.cc
@@ -309,10 +309,12 @@ InternalPageInfoPopupView::InternalPageInfoPopupView(
SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal, kSpacing,
kSpacing, kSpacing));
set_margins(gfx::Insets());
- views::ImageView* icon_view = new NonAccessibleImageView();
- ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
- icon_view->SetImage(rb.GetImageSkiaNamed(icon));
- AddChildView(icon_view);
+ if (LayoutDelegate::Get()->ShouldShowWindowIcon()) {
+ views::ImageView* icon_view = new NonAccessibleImageView();
+ ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
+ icon_view->SetImage(rb.GetImageSkiaNamed(icon));
+ AddChildView(icon_view);
+ }
views::Label* label = new views::Label(l10n_util::GetStringUTF16(text));
label->SetMultiLine(true);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698