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

Unified Diff: chrome/browser/chromeos/extensions/file_manager/file_browser_handlers.h

Issue 23857004: file_manager: Unify the file task selection logic (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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/chromeos/extensions/file_manager/file_browser_handlers.h
diff --git a/chrome/browser/chromeos/extensions/file_manager/file_browser_handlers.h b/chrome/browser/chromeos/extensions/file_manager/file_browser_handlers.h
index 5c446ee56724ad31e9a8696fbff9c2d73e4d80c3..b969ed37b8738db4dae43200d9c5b1c82ef9f770 100644
--- a/chrome/browser/chromeos/extensions/file_manager/file_browser_handlers.h
+++ b/chrome/browser/chromeos/extensions/file_manager/file_browser_handlers.h
@@ -43,30 +43,12 @@ typedef std::vector<const FileBrowserHandler*> FileBrowserHandlerList;
// extensions.
bool IsFallbackFileBrowserHandler(const file_tasks::TaskDescriptor& task);
-// Finds file browser handlers set as default from |common_tasks| for
-// |file_list|. If no handlers are set as default, choose the the firstly
-// found fallback handler as default.
-FileBrowserHandlerList FindDefaultFileBrowserHandlers(
- const PrefService& pref_service,
- const std::vector<base::FilePath>& file_list,
- const FileBrowserHandlerList& common_tasks);
-
// Returns the list of file browser handlers that can open all files in
// |file_list|.
-FileBrowserHandlerList FindCommonFileBrowserHandlers(
+FileBrowserHandlerList FindFileBrowserHandlers(
Profile* profile,
const std::vector<GURL>& file_list);
-// Finds a file browser handler for a file whose URL is |url| and whose path
-// is |path|. Returns the default handler if one is defined (The default
-// handler is the one that is assigned to the file manager task button by
-// default). If the default handler is not found, tries to match the url with
-// one of the file browser handlers.
-const FileBrowserHandler* FindFileBrowserHandlerForURLAndPath(
- Profile* profile,
- const GURL& url,
- const base::FilePath& path);
-
// Executes a file browser handler specified by |extension| of the given
// action ID for |file_urls|. Returns false if undeclared handlers are
// found. |done| is on completion. See also the comment at ExecuteFileTask()

Powered by Google App Engine
This is Rietveld 408576698