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

Unified Diff: chrome/browser/extensions/api/omnibox/omnibox_api.h

Issue 2576833002: Make some updates to extension iconography. (Closed)
Patch Set: add test Created 4 years 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/api/omnibox/omnibox_api.h
diff --git a/chrome/browser/extensions/api/omnibox/omnibox_api.h b/chrome/browser/extensions/api/omnibox/omnibox_api.h
index 9d88a1a9e6a742d878e3846904e8ffa3204bfb99..5fbec3e645d9206f228c3ae25890f6bcf93309c5 100644
--- a/chrome/browser/extensions/api/omnibox/omnibox_api.h
+++ b/chrome/browser/extensions/api/omnibox/omnibox_api.h
@@ -95,13 +95,10 @@ class OmniboxAPI : public BrowserContextKeyedAPI,
// KeyedService implementation.
void Shutdown() override;
- // Returns the icon to display in the omnibox for the given extension.
+ // Returns the icon to display in the location bar or omnibox popup for the
+ // given extension.
gfx::Image GetOmniboxIcon(const std::string& extension_id);
- // Returns the icon to display in the omnibox popup window for the given
- // extension.
- gfx::Image GetOmniboxPopupIcon(const std::string& extension_id);
-
private:
friend class BrowserContextKeyedAPIFactory<OmniboxAPI>;
@@ -136,7 +133,6 @@ class OmniboxAPI : public BrowserContextKeyedAPI,
// Keeps track of favicon-sized omnibox icons for extensions.
ExtensionIconManager omnibox_icon_manager_;
- ExtensionIconManager omnibox_popup_icon_manager_;
std::unique_ptr<TemplateURLService::Subscription> template_url_sub_;

Powered by Google App Engine
This is Rietveld 408576698