Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(48)

Unified Diff: chrome/browser/chromeos/arc/fileapi/arc_documents_provider_root.cc

Issue 2661003003: mediaview: Recognize more MIME types. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/arc/fileapi/arc_documents_provider_root.cc
diff --git a/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_root.cc b/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_root.cc
index 1eb533f1bfa5a51c9ef93e715529d020b948bd5f..8264bcad61dcee21f2018dcbc5eb01a53d092985 100644
--- a/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_root.cc
+++ b/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_root.cc
@@ -18,7 +18,6 @@
#include "chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h"
#include "chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_util.h"
#include "content/public/browser/browser_thread.h"
-#include "net/base/mime_util.h"
#include "url/gurl.h"
using content::BrowserThread;
@@ -53,17 +52,13 @@ base::FilePath::StringType GetFileNameForDocument(
base::ToLowerASCII(base::FilePath(filename).Extension());
if (!extension.empty())
extension = extension.substr(1); // Strip the leading dot.
- std::vector<base::FilePath::StringType> possible_extensions;
- net::GetExtensionsForMimeType(document->mime_type, &possible_extensions);
+ std::vector<base::FilePath::StringType> possible_extensions =
+ GetExtensionsForArcMimeType(document->mime_type);
if (!possible_extensions.empty() &&
std::find(possible_extensions.begin(), possible_extensions.end(),
extension) == possible_extensions.end()) {
- base::FilePath::StringType new_extension;
- if (!net::GetPreferredExtensionForMimeType(document->mime_type,
- &new_extension)) {
- new_extension = possible_extensions[0];
- }
- filename = base::FilePath(filename).AddExtension(new_extension).value();
+ filename =
+ base::FilePath(filename).AddExtension(possible_extensions[0]).value();
}
return filename;
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/fileapi/arc_documents_provider_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698