chrome/browser/chromeos/extensions/file_handler_util.h - Issue 9692049: Gave native File Browser file handlers higher priority than 3rd party handlers.

Unified Diff: chrome/browser/chromeos/extensions/file_handler_util.h

Issue 9692049: Gave native File Browser file handlers higher priority than 3rd party handlers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 8 years, 9 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

« no previous file with comments | « chrome/browser/chromeos/extensions/file_browser_private_api.cc ('k') | chrome/browser/chromeos/extensions/file_handler_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/chromeos/extensions/file_handler_util.h

diff --git a/chrome/browser/chromeos/extensions/file_handler_util.h b/chrome/browser/chromeos/extensions/file_handler_util.h

index f17fd01a37e1ed840de3610846c4efae9d7d0c35..462604b15013fc3dd5b51dbf24246f077624aec5 100644

--- a/chrome/browser/chromeos/extensions/file_handler_util.h

+++ b/chrome/browser/chromeos/extensions/file_handler_util.h

@@ -42,10 +42,12 @@ typedef std::vector<LastUsedHandler> LastUsedHandlerList;

bool FindCommonTasks(Profile* profile,

const std::vector<GURL>& files_list,

+ const std::string& privileged_extension_id,

SeRya 2012/03/14 11:44:57 Could this parameter different from kFileBrowserDo

Vladislav Kaznacheev 2012/03/14 12:01:05 It is not about flexibility but about dependencies

SeRya 2012/03/14 12:19:22 Could we resolve the dependency issue not touching

Vladislav Kaznacheev 2012/03/14 12:28:07 Checking for component extensions is not enough. T

dgozman 2012/03/14 13:31:11 AFAIK, Picasa Uploader is not a component extensio

LastUsedHandlerList* named_action_list);

bool GetDefaultTask(Profile* profile,

const GURL& url,

+ const std::string& privileged_extension_id,

const FileBrowserHandler** handler);

class FileTaskExecutor : public base::RefCountedThreadSafe<FileTaskExecutor> {