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

Unified Diff: components/toolbar/test_toolbar_model.h

Issue 2644903004: Move around more vector icons. (Closed)
Patch Set: fix comment Created 3 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
« no previous file with comments | « components/toolbar/BUILD.gn ('k') | components/toolbar/test_toolbar_model.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/toolbar/test_toolbar_model.h
diff --git a/components/toolbar/test_toolbar_model.h b/components/toolbar/test_toolbar_model.h
index 255ac00f63c1c8aab10b4a152a6c821de2e2e465..01daf608d4628854d1a1a5cb6cacde9a8031e37d 100644
--- a/components/toolbar/test_toolbar_model.h
+++ b/components/toolbar/test_toolbar_model.h
@@ -13,7 +13,7 @@
#include "components/toolbar/toolbar_model.h"
namespace gfx {
-enum class VectorIconId;
+struct VectorIcon;
}
// A ToolbarModel that is backed by instance variables, which are initialized
@@ -27,7 +27,7 @@ class TestToolbarModel : public ToolbarModel {
GURL GetURL() const override;
security_state::SecurityLevel GetSecurityLevel(
bool ignore_editing) const override;
- gfx::VectorIconId GetVectorIcon() const override;
+ const gfx::VectorIcon& GetVectorIcon() const override;
base::string16 GetSecureVerboseText() const override;
base::string16 GetEVCertName() const override;
bool ShouldDisplayURL() const override;
@@ -37,7 +37,7 @@ class TestToolbarModel : public ToolbarModel {
void set_security_level(security_state::SecurityLevel security_level) {
security_level_ = security_level;
}
- void set_icon(gfx::VectorIconId icon) { icon_ = icon; }
+ void set_icon(const gfx::VectorIcon& icon) { icon_ = &icon; }
void set_ev_cert_name(const base::string16& ev_cert_name) {
ev_cert_name_ = ev_cert_name;
}
@@ -49,7 +49,7 @@ class TestToolbarModel : public ToolbarModel {
base::string16 text_;
GURL url_;
security_state::SecurityLevel security_level_;
- gfx::VectorIconId icon_;
+ const gfx::VectorIcon* icon_ = nullptr;
base::string16 ev_cert_name_;
bool should_display_url_;
« no previous file with comments | « components/toolbar/BUILD.gn ('k') | components/toolbar/test_toolbar_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698