Have media gallery (through native media file util) use MIME sniffer

webkit/fileapi/media/device_media_async_file_util.cc

Index: webkit/fileapi/media/device_media_async_file_util.cc
diff --git a/webkit/fileapi/media/device_media_async_file_util.cc b/webkit/fileapi/media/device_media_async_file_util.cc
index 6235696ded0ad7a8fa692d1d1da720dfc779978b..94a27de8f05938887d145e013ebe4aae128f23c5 100644
--- a/webkit/fileapi/media/device_media_async_file_util.cc
+++ b/webkit/fileapi/media/device_media_async_file_util.cc
@@ -17,6 +17,8 @@
 #include "webkit/fileapi/media/mtp_device_async_delegate.h"
 #include "webkit/fileapi/media/mtp_device_file_system_config.h"
 #include "webkit/fileapi/media/mtp_device_map_service.h"
+#include "webkit/fileapi/media/native_media_file_util.h"
+#include "webkit/fileapi/native_file_util.h"
 
 namespace fileapi {
 
@@ -61,6 +63,15 @@ void CreateSnapshotFileOnBlockingPool(
   }
 }
 
+void CheckMediaFile(
+    const base::FilePath& file_path,
+    base::PlatformFileError* p_error) {

[vandebo (ex-Chrome), 2013/03/29 21:53:20]

nit: error, not p_error in all your change.

[Kevin Bailey, 2013/04/03 18:05:07]

Done.

+  *p_error = NativeMediaFileUtil::IsMediaFile(file_path);
+  if (*p_error == base::PLATFORM_FILE_ERROR_SECURITY)
+    // It exists but we reject it. Try to clean up.
+    NativeFileUtil::DeleteFile(file_path);
+}
+
 }  // namespace
 
 DeviceMediaAsyncFileUtil::~DeviceMediaAsyncFileUtil() {
@@ -309,12 +320,30 @@ void DeviceMediaAsyncFileUtil::OnReadDirectoryError(
 }
 
 void DeviceMediaAsyncFileUtil::OnDidCreateSnapshotFile(
+    FileSystemOperationContext* context,
     const AsyncFileUtil::CreateSnapshotFileCallback& callback,
     const base::PlatformFileInfo& file_info,
     const base::FilePath& platform_path) {
-  if (!callback.is_null())
-    callback.Run(base::PLATFORM_FILE_OK, file_info, platform_path,
-                 kSnapshotFileTemporary);
+  base::PlatformFileError* p_error = new base::PlatformFileError;
+  context->file_system_context()->task_runners()->media_task_runner()->
+      PostTaskAndReply(
+          FROM_HERE,
+          base::Bind(&CheckMediaFile, platform_path,
+                     base::Unretained(p_error)),
+          base::Bind(&DeviceMediaAsyncFileUtil::OnDidCheckMediaRunTask,
+                     weak_ptr_factory_.GetWeakPtr(),
+                     callback, file_info, platform_path,
+                     base::Owned(p_error)));
+}
+
+void DeviceMediaAsyncFileUtil::OnDidCheckMediaRunTask(
+    const AsyncFileUtil::CreateSnapshotFileCallback& callback,
+    const base::PlatformFileInfo& file_info,
+    const base::FilePath& platform_path,
+    base::PlatformFileError* p_error) {
+  if (!callback.is_null()) {
+    callback.Run(*p_error, file_info, platform_path, kSnapshotFileTemporary);
+  }
 }
 
 void DeviceMediaAsyncFileUtil::OnCreateSnapshotFileError(
@@ -345,6 +374,7 @@ void DeviceMediaAsyncFileUtil::OnSnapshotFileCreatedRunTask(
       *snapshot_file_path,
       base::Bind(&DeviceMediaAsyncFileUtil::OnDidCreateSnapshotFile,
                  weak_ptr_factory_.GetWeakPtr(),
+                 context,
                  callback),
       base::Bind(&DeviceMediaAsyncFileUtil::OnCreateSnapshotFileError,
                  weak_ptr_factory_.GetWeakPtr(),