mediaview: Recognize more MIME types.

chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h

Index: chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h
diff --git a/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h b/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h
index 1ba3583e6a37d3d1238ebbf65f38d0feeead89ba..c419451d1ac3af03d9455db7598c84de7e9355b5 100644
--- a/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h
+++ b/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h
@@ -8,6 +8,7 @@
 #define CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_DOCUMENTS_PROVIDER_UTIL_H_
 
 #include <string>
+#include <vector>
 
 #include "base/files/file_path.h"
 
@@ -61,6 +62,14 @@ bool ParseDocumentsProviderUrl(const storage::FileSystemURL& url,
 GURL BuildDocumentUrl(const std::string& authority,
                       const std::string& document_id);
 
+// Similar to net::GetExtensionsForMimeType(), but this covers more MIME types
+// used in Android.
+// Returns an empty vector if the MIME type is not known.
+// If the returned vector is not empty, the first extension is the preferred
+// extension.
+std::vector<base::FilePath::StringType> GetExtensionsForMimeTypeForArc(

[hidehiko, 2017/01/27 07:39:44]

Optional: two For may be annoying? How about
- GetExtensionsForArcMimeType
- GetExtensionsForMineType (defined in arc namespace, so ForArc may be
redundant).

[Shuhei Takahashi, 2017/01/27 09:17:14]

OK, let's call it GetExtensionsForArcMimeType.

(I did not choose GetExtensionsForMineType() because it's too misleading)

[hidehiko, 2017/01/27 09:39:00]

sgtm

+    const std::string& mime_type);
+
 }  // namespace arc
 
 #endif  // CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_DOCUMENTS_PROVIDER_UTIL_H_