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

Unified Diff: extensions/browser/api/management/management_api.cc

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: extensions/browser/api/management/management_api.cc
diff --git a/extensions/browser/api/management/management_api.cc b/extensions/browser/api/management/management_api.cc
index 0c9d32584a3b7d456f0e2087ee2f1b718970ece5..18f94189620adb1a0bb58b74cdd42ef885bf74a8 100644
--- a/extensions/browser/api/management/management_api.cc
+++ b/extensions/browser/api/management/management_api.cc
@@ -162,9 +162,8 @@ management::ExtensionInfo CreateExtensionInfo(
for (icon_iter = icons.begin(); icon_iter != icons.end(); ++icon_iter) {
management::IconInfo icon_info;
icon_info.size = icon_iter->first;
- GURL url =
- delegate->GetIconURL(&extension, icon_info.size,
- ExtensionIconSet::MATCH_EXACTLY, false, nullptr);
+ GURL url = delegate->GetIconURL(&extension, icon_info.size,
+ ExtensionIconSet::MATCH_EXACTLY, false);
icon_info.url = url.spec();
info.icons->push_back(std::move(icon_info));
}

Powered by Google App Engine
This is Rietveld 408576698