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

Unified Diff: chrome/browser/chromeos/file_manager/file_tasks.cc

Issue 2576833002: Make some updates to extension iconography. (Closed)
Patch Set: devlin review 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
« no previous file with comments | « no previous file | chrome/browser/extensions/api/management/chrome_management_api_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/file_manager/file_tasks.cc
diff --git a/chrome/browser/chromeos/file_manager/file_tasks.cc b/chrome/browser/chromeos/file_manager/file_tasks.cc
index 1079d744510de8fc0b46e8bc79b8faaa34266bc7..ffd98c0564ef2f5e790f18230ba41aaa8a71e727 100644
--- a/chrome/browser/chromeos/file_manager/file_tasks.cc
+++ b/chrome/browser/chromeos/file_manager/file_tasks.cc
@@ -503,8 +503,7 @@ void FindFileHandlerTasks(Profile* profile,
GURL best_icon = extensions::ExtensionIconSource::GetIconURL(
extension, drive::util::kPreferredIconSize,
ExtensionIconSet::MATCH_BIGGER,
- false, // grayscale
- NULL); // exists
+ false); // grayscale
// If file handler doesn't match as good match, regards it as generic file
// handler.
@@ -559,11 +558,9 @@ void FindFileBrowserHandlerTasks(
// TODO(zelidrag): Figure out how to expose icon URL that task defined in
// manifest instead of the default extension icon.
const GURL icon_url = extensions::ExtensionIconSource::GetIconURL(
- extension,
- extension_misc::EXTENSION_ICON_BITTY,
+ extension, extension_misc::EXTENSION_ICON_BITTY,
ExtensionIconSet::MATCH_BIGGER,
- false, // grayscale
- NULL); // exists
+ false); // grayscale
result_list->push_back(FullTaskDescriptor(
TaskDescriptor(extension_id, file_tasks::TASK_TYPE_FILE_BROWSER_HANDLER,
« no previous file with comments | « no previous file | chrome/browser/extensions/api/management/chrome_management_api_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698