Files.app: Fix a call of the default task picker.

chrome/browser/chromeos/extensions/file_manager/private_api_tasks.cc

Index: chrome/browser/chromeos/extensions/file_manager/private_api_tasks.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/private_api_tasks.cc b/chrome/browser/chromeos/extensions/file_manager/private_api_tasks.cc
index fa4573077533334e181a5089ab5ee919691517d8..b0244ef87237f0a272ce171727a3780910167794 100644
--- a/chrome/browser/chromeos/extensions/file_manager/private_api_tasks.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/private_api_tasks.cc
@@ -47,9 +47,11 @@ std::set<std::string> GetUniqueMimeTypes(
     const std::vector<std::string>& mime_type_list) {
   std::set<std::string> mime_types;
   for (size_t i = 0; i < mime_type_list.size(); ++i) {
-    std::string mime_type;
-    // We'll skip empty MIME types.
-    if (!mime_type.empty())
+    const std::string mime_type = mime_type_list[i];
+    // We'll skip empty MIME types and existing MIME types.
+    if (!mime_type.empty() ||
+        std::find(mime_types.begin(), mime_types.end(), mime_type) ==

[yoshiki, 2013/12/10 07:00:29]

We don't need to check duplication explicitly since we are using std::set.

[hirono, 2013/12/10 10:05:33]

Oops, I missed it. Thanks!

+        mime_types.end())
       mime_types.insert(mime_type);
   }
   return mime_types;