Replace sets with vectors when storing file handlers.

chrome/browser/chromeos/extensions/file_manager/file_handler_util.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_FILE_HANDLER_UTIL_H_
 #define CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_FILE_HANDLER_UTIL_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 10 matching lines @@
 namespace extensions {
 class Extension;
 }  // namespace extensions
 
 namespace fileapi {
 class FileSystemURL;
 }
 
 namespace file_handler_util {
 
+// Tasks are stored as a vector in order of priorities.
+typedef std::vector<const FileBrowserHandler*> FileBrowserHandlerList;
+
 // Specifies the task type for a task id that represents some file action, Drive
 // action, or Web Intent action.
 extern const char kTaskFile[];
 extern const char kTaskDrive[];
 extern const char kTaskApp[];
 
 void UpdateFileHandlerUsageStats(Profile* profile, const std::string& task_id);
 
 // Update the default file handler for the given sets of suffixes and MIME
 // types.
@@ skipping 22 matching lines @@
                        const std::string& action_id);
 
 // Extracts action, type and extension id bound to the file task. Either
 // |target_extension_id| or |action_id| are allowed to be NULL if caller isn't
 // interested in those values.  Returns false on failure to parse.
 bool CrackTaskID(const std::string& task_id,
                  std::string* target_extension_id,
                  std::string* task_type,
                  std::string* action_id);
 
-// This generates a list of default tasks (tasks set as default by the user in

[kinaba, 2013/05/24 07:34:13]

nit: probably this "set" is a past participle of the verb "set" so it should be
kept as is.

[mtomasz, 2013/05/24 07:49:45]

Done.

+// This generates a list of default tasks (tasks list as default by the user in
 // prefs) from the |common_tasks|.
 void FindDefaultTasks(Profile* profile,
                       const std::vector<base::FilePath>& files_list,
-                      const std::set<const FileBrowserHandler*>& common_tasks,
-                      std::set<const FileBrowserHandler*>* default_tasks);
+                      const FileBrowserHandlerList& common_tasks,
+                      FileBrowserHandlerList* default_tasks);
 
 // This generates list of tasks common for all files in |file_list|.
 bool FindCommonTasks(Profile* profile,
                      const std::vector<GURL>& files_list,
-                     std::set<const FileBrowserHandler*>* common_tasks);
+                     FileBrowserHandlerList* common_tasks);
 
 // Finds a task for a file whose URL is |url| and whose path is |path|.
 // Returns default task if one is defined (The default task is the task that is
 // assigned to file browser task button by default). If default task is not
 // found, tries to match the url with one of the builtin tasks.
 bool GetTaskForURLAndPath(Profile* profile,
                           const GURL& url,
                           const base::FilePath& path,
                           const FileBrowserHandler** handler);
 
@@ skipping 54 matching lines @@
 
   Profile* profile_;
   const GURL source_url_;
   const std::string file_browser_id_;
   const std::string extension_id_;
 };
 
 }  // namespace file_handler_util
 
 #endif  // CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_FILE_HANDLER_UTIL_H_