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

Unified Diff: chrome/browser/ui/webui/options/search_engine_manager_handler.cc

Issue 11786003: Move Icons out of Extension class (Closed) Base URL: http://git.chromium.org/chromium/src.git@dc_unref_browser_action
Patch Set: Created 7 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
Index: chrome/browser/ui/webui/options/search_engine_manager_handler.cc
diff --git a/chrome/browser/ui/webui/options/search_engine_manager_handler.cc b/chrome/browser/ui/webui/options/search_engine_manager_handler.cc
index 5c0c1fbb71972eaa7eddb304572022f4b6224b30..cbb17418ea5f9ad4889649521c767e4bafb24105 100644
--- a/chrome/browser/ui/webui/options/search_engine_manager_handler.cc
+++ b/chrome/browser/ui/webui/options/search_engine_manager_handler.cc
@@ -14,6 +14,7 @@
#include "chrome/browser/search_engines/template_url_service.h"
#include "chrome/browser/ui/search_engines/keyword_editor_controller.h"
#include "chrome/browser/ui/search_engines/template_url_table_model.h"
+#include "chrome/common/extensions/api/icons/icons_handler.h"
#include "chrome/common/extensions/api/omnibox/omnibox_handler.h"
#include "chrome/common/extensions/extension.h"
#include "chrome/common/url_constants.h"
@@ -173,7 +174,8 @@ base::DictionaryValue* SearchEngineManagerHandler::CreateDictionaryForExtension(
dict->SetString("displayName", extension.name());
dict->SetString("keyword",
extensions::OmniboxInfo::GetKeyword(&extension));
- GURL icon = extension.GetIconURL(16, ExtensionIconSet::MATCH_BIGGER);
+ GURL icon = extensions::IconsInfo::GetIconURL(
+ &extension, 16, ExtensionIconSet::MATCH_BIGGER);
dict->SetString("iconURL", icon.spec());
dict->SetString("url", string16());
return dict;

Powered by Google App Engine
This is Rietveld 408576698