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

Unified Diff: chrome/browser/ui/views/website_settings/permission_prompt_impl.cc

Issue 2685683004: Move more vector icons to more appropriate directories. (Closed)
Patch Set: rebase Created 3 years, 10 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 | « chrome/browser/ui/views/download/download_started_animation_views.cc ('k') | media/remoting/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/website_settings/permission_prompt_impl.cc
diff --git a/chrome/browser/ui/views/website_settings/permission_prompt_impl.cc b/chrome/browser/ui/views/website_settings/permission_prompt_impl.cc
index eeb72546ed18ad485576d82e959ff3d374e9d1a8..9b8252cffc26a66b284591b3959ed7b0e2f298f8 100644
--- a/chrome/browser/ui/views/website_settings/permission_prompt_impl.cc
+++ b/chrome/browser/ui/views/website_settings/permission_prompt_impl.cc
@@ -233,11 +233,9 @@ PermissionsBubbleDialogDelegateView::PermissionsBubbleDialogDelegateView(
layout_delegate->GetMetric(
LayoutDelegate::Metric::RELATED_LABEL_HORIZONTAL_SPACING)));
views::ImageView* icon = new views::ImageView();
- gfx::VectorIconId vector_id = requests[index]->GetIconId();
- if (vector_id != gfx::VectorIconId::VECTOR_ICON_NONE) {
- icon->SetImage(
- gfx::CreateVectorIcon(vector_id, kIconSize, gfx::kChromeIconGrey));
- }
+ const gfx::VectorIcon& vector_id = requests[index]->GetIconId();
+ icon->SetImage(
+ gfx::CreateVectorIcon(vector_id, kIconSize, gfx::kChromeIconGrey));
icon->SetTooltipText(base::string16()); // Redundant with the text fragment
label_container->AddChildView(icon);
views::Label* label =
« no previous file with comments | « chrome/browser/ui/views/download/download_started_animation_views.cc ('k') | media/remoting/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698