ChromeOS: Use file extensions in Files app to decide which apps to use.

chrome/browser/extensions/api/file_handlers/app_file_handler_util.h

Index: chrome/browser/extensions/api/file_handlers/app_file_handler_util.h
diff --git a/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h b/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h
index ad7d4b7465a63ef23193ff354995a99efdd58d06..ac0939667c7921eac7859a32d576980e2b4d65ab 100644
--- a/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h
+++ b/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h
@@ -7,6 +7,7 @@
 
 #include <set>
 #include <string>
+#include <utility>
 #include <vector>
 
 #include "chrome/common/extensions/api/file_handlers/file_handlers_parser.h"
@@ -19,6 +20,8 @@ namespace extensions {
 // TODO(benwells): move this to platform_apps namespace.
 namespace app_file_handler_util {
 
+typedef std::set<std::pair<std::string, base::FilePath> > MimeTypeAndPathSet;

[Matt Giuca, 2013/05/07 05:39:59]

Document this type.

Also I would swap the order of the pair, because then it's like a key:value
mapping where the filepath is the key, and mime type is the value. (It seems
more natural to me to have the key on the left side of the pair.)

[Sam McNally, 2013/05/07 06:15:27]

Done.

+
 // Returns the file handler with the specified |handler_id|, or NULL if there
 // is no such handler.
 const FileHandlerInfo* FileHandlerForId(const Extension& app,
@@ -32,16 +35,13 @@ const FileHandlerInfo* FirstFileHandlerForFile(
     const base::FilePath& path);
 
 std::vector<const FileHandlerInfo*>
-FindFileHandlersForMimeTypes(const Extension& extension,
-                             const std::set<std::string>& mime_types);
+FindFileHandlers(const Extension& extension,
+                 const MimeTypeAndPathSet& files);
 
 bool FileHandlerCanHandleFile(
     const FileHandlerInfo& handler,
     const std::string& mime_type,
     const base::FilePath& path);
-bool FileHandlerCanHandleFileWithMimeType(
-    const FileHandlerInfo& handler,
-    const std::string& mime_type);
 
 // Represents a file entry that a user has given an extension permission to
 // access. Intended to be persisted to disk (in the Preferences file), so should