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

Unified Diff: chrome/browser/ui/content_settings/content_setting_image_model.h

Issue 2721823002: Update content settings to new vector icon format. (Closed)
Patch Set: 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/app/vector_icons/web.icon ('k') | chrome/browser/ui/content_settings/content_setting_image_model.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/content_settings/content_setting_image_model.h
diff --git a/chrome/browser/ui/content_settings/content_setting_image_model.h b/chrome/browser/ui/content_settings/content_setting_image_model.h
index c6e16a35051e03cfd9d9dd3192d44fd0c85f1660..490f30e9ccc54dc324e1aed09f7a3774265c6ad8 100644
--- a/chrome/browser/ui/content_settings/content_setting_image_model.h
+++ b/chrome/browser/ui/content_settings/content_setting_image_model.h
@@ -18,7 +18,7 @@ class WebContents;
}
namespace gfx {
-enum class VectorIconId;
+struct VectorIcon;
}
// This model provides data (icon ids and tooltip) for the content setting icons
@@ -72,9 +72,9 @@ class ContentSettingImageModel {
protected:
ContentSettingImageModel();
- void set_icon_by_vector_id(gfx::VectorIconId id, gfx::VectorIconId badge_id) {
- icon_id_ = id;
- icon_badge_id_ = badge_id;
+ void set_icon(const gfx::VectorIcon& icon, const gfx::VectorIcon& badge) {
+ icon_ = &icon;
+ icon_badge_ = &badge;
}
void set_visible(bool visible) { is_visible_ = visible; }
@@ -86,8 +86,8 @@ class ContentSettingImageModel {
private:
bool is_visible_;
- gfx::VectorIconId icon_id_;
- gfx::VectorIconId icon_badge_id_;
+ const gfx::VectorIcon* icon_;
+ const gfx::VectorIcon* icon_badge_;
int explanatory_string_id_;
base::string16 tooltip_;
« no previous file with comments | « chrome/app/vector_icons/web.icon ('k') | chrome/browser/ui/content_settings/content_setting_image_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698