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

Unified Diff: chrome/browser/extensions/extension_action_icon_factory_unittest.cc

Issue 2537263003: [Extensions] Fix lifetime bug in ExtensionAction/IconImage (Closed)
Patch Set: Created 4 years, 1 month 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/extensions/extension_action_icon_factory_unittest.cc
diff --git a/chrome/browser/extensions/extension_action_icon_factory_unittest.cc b/chrome/browser/extensions/extension_action_icon_factory_unittest.cc
index 275b802da132ea0f11cc17d10eb132bd668db167..34ea3c6c5c7a7a859df4307ac58dc1f64307a2ad 100644
--- a/chrome/browser/extensions/extension_action_icon_factory_unittest.cc
+++ b/chrome/browser/extensions/extension_action_icon_factory_unittest.cc
@@ -275,19 +275,20 @@ TEST_P(ExtensionActionIconFactoryTest, DefaultIcon) {
ASSERT_FALSE(browser_action->default_icon());
ASSERT_TRUE(browser_action->GetExplicitlySetIcon(0 /*tab id*/).IsEmpty());
+ scoped_refptr<const Extension> extension_with_icon =
+ CreateExtension("browser_action_with_icon", Manifest::INVALID_LOCATION);
+ ASSERT_TRUE(extension_with_icon);
+
int icon_size = ExtensionAction::ActionIconSize();
gfx::Image default_icon =
- EnsureImageSize(LoadIcon("browser_action/no_icon/icon.png"), icon_size);
+ EnsureImageSize(LoadIcon("browser_action_with_icon/icon.png"), icon_size);
ASSERT_FALSE(default_icon.IsEmpty());
- std::unique_ptr<ExtensionIconSet> default_icon_set(new ExtensionIconSet());
- default_icon_set->Add(icon_size, "icon.png");
-
- browser_action->SetDefaultIconForTest(std::move(default_icon_set));
+ browser_action = GetBrowserAction(*extension_with_icon);
ASSERT_TRUE(browser_action->default_icon());
ExtensionActionIconFactory icon_factory(
- profile(), extension.get(), browser_action, this);
+ profile(), extension_with_icon.get(), browser_action, this);
gfx::Image icon = icon_factory.GetIcon(0);
« no previous file with comments | « chrome/browser/extensions/extension_action_icon_factory.cc ('k') | chrome/browser/extensions/extension_action_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698