Add isGenericFileHandler property to FileTask.

chrome/browser/chromeos/file_manager/file_tasks.cc

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 238a29eee0f59909d9e97c557d354d8497e98ef5..a648b27cf53f78b9d24a86a34cd751bc8c1c147e 100644
--- a/chrome/browser/chromeos/file_manager/file_tasks.cc
+++ b/chrome/browser/chromeos/file_manager/file_tasks.cc
@@ -127,17 +127,25 @@ bool IsFallbackFileHandler(const file_tasks::TaskDescriptor& task) {
   return false;
 }
 
+bool IsGenericFileHandler(const extensions::FileHandlerInfo& fileHandlerInfo) {
+  return fileHandlerInfo.extensions.count("*") > 0 ||
+      fileHandlerInfo.types.count("*") > 0 ||
+      fileHandlerInfo.types.count("*/*") > 0;
+}
+
 }  // namespace
 
 FullTaskDescriptor::FullTaskDescriptor(
     const TaskDescriptor& task_descriptor,
     const std::string& task_title,
     const GURL& icon_url,
-    bool is_default)
+    bool is_default,
+    bool is_generic_file_handler)
     : task_descriptor_(task_descriptor),
       task_title_(task_title),
       icon_url_(icon_url),
-      is_default_(is_default) {
+      is_default_(is_default),
+      is_generic_file_handler_(is_generic_file_handler) {
 }
 
 void UpdateDefaultTask(PrefService* pref_service,
@@ -353,7 +361,8 @@ void FindDriveAppTasks(
         FullTaskDescriptor(descriptor,
                            app_info.app_name,
                            icon_url,
-                           false /* is_default */));
+                           false /* is_default */,
+                           false /* is_generic_file_handler */));
   }
 }
 
@@ -366,6 +375,7 @@ void FindFileHandlerTasks(
 
   const extensions::ExtensionSet& enabled_extensions =
       extensions::ExtensionRegistry::Get(profile)->enabled_extensions();
+
   for (extensions::ExtensionSet::const_iterator iter =
            enabled_extensions.begin();
        iter != enabled_extensions.end();
@@ -391,8 +401,18 @@ void FindFileHandlerTasks(
     if (file_handlers.empty())
       continue;
 
-    // Only show the first matching handler from each app.
-    const extensions::FileHandlerInfo* file_handler = file_handlers.front();
+    // Only show the first matching non-generic handler from each app.

[fukino, 2014/10/30 07:56:33]

I think this comment is not correct.
Non-generic handlers are preferred, but generic handlers can be shown when
there's no non-generic handlers, right?

[yawano, 2014/10/30 08:06:18]

Yes, the comment was wrong. I fixed it.

+    const extensions::FileHandlerInfo* file_handler = nullptr;
+    for (auto handler : file_handlers) {
+      if (!IsGenericFileHandler(*handler)) {
+        file_handler = handler;
+        break;
+      }
+    }
+    if (file_handler == nullptr) {
+      file_handler = file_handlers.front();
+    }
+
     std::string task_id = file_tasks::MakeTaskID(
         extension->id(), file_tasks::TASK_TYPE_FILE_HANDLER, file_handler->id);
 
@@ -409,7 +429,8 @@ void FindFileHandlerTasks(
                                           file_handler->id),
                            extension->name(),
                            best_icon,
-                           false /* is_default */));
+                           false /* is_default */,
+                           IsGenericFileHandler(*file_handler)));
   }
 }
 
@@ -451,7 +472,8 @@ void FindFileBrowserHandlerTasks(
                        handler->id()),
         handler->title(),
         icon_url,
-        false /* is_default */));
+        false /* is_default */,
+        false /* is_generic_file_handler */));
   }
 }