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

webkit/fileapi/media/native_media_file_util.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef WEBKIT_FILEAPI_MEDIA_NATIVE_MEDIA_FILE_UTIL_H_
 #define WEBKIT_FILEAPI_MEDIA_NATIVE_MEDIA_FILE_UTIL_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "webkit/fileapi/isolated_file_util.h"
 #include "webkit/storage/webkit_storage_export.h"
@@ skipping 40 matching lines @@
         const base::FilePath& src_file_path,
         const FileSystemURL& dest_url) OVERRIDE;
   virtual base::PlatformFileError DeleteFile(
       FileSystemOperationContext* context,
       const FileSystemURL& url) OVERRIDE;
   virtual base::PlatformFileError GetFileInfo(
       FileSystemOperationContext* context,
       const FileSystemURL& url,
       base::PlatformFileInfo* file_info,
       base::FilePath* platform_path) OVERRIDE;
+  // Uses the MIME sniffer code, which actually looks into the file,
+  // to determine if it is really a media file (to avoid exposing
+  // non-media files with a media file extension.)
+  static base::PlatformFileError IsMediaFile(const base::FilePath& path);

[vandebo (ex-Chrome), 2013/03/25 19:02:11]

All the othe rmethods are implementations of IsolatedFileUtil, so they should
stay together.  Put IsMediaFile before or after the block of overriding
functions and correspondly move the implementation.

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

Done.

+  virtual base::PlatformFileError CreateSnapshotFile(
+      FileSystemOperationContext* context,
+      const FileSystemURL& url,
+      base::PlatformFileInfo* file_info,
+      base::FilePath* platform_path,
+      SnapshotFilePolicy* snapshot_policy) OVERRIDE;
 
  private:
   // Like GetLocalFilePath(), but always take media_path_filter() into
   // consideration. If the media_path_filter() check fails, return
   // PLATFORM_FILE_ERROR_SECURITY. |local_file_path| does not have to exist.
   base::PlatformFileError GetFilteredLocalFilePath(
       FileSystemOperationContext* context,
       const FileSystemURL& file_system_url,
       base::FilePath* local_file_path);
 
   // Like GetLocalFilePath(), but if the file does not exist, then return
   // |failure_error|.
   // If |local_file_path| is a file, then take media_path_filter() into
   // consideration.
   // If the media_path_filter() check fails, return |failure_error|.
   // If |local_file_path| is a directory, return PLATFORM_FILE_OK.
   base::PlatformFileError GetFilteredLocalFilePathForExistingFileOrDirectory(
       FileSystemOperationContext* context,
       const FileSystemURL& file_system_url,
       base::PlatformFileError failure_error,
       base::FilePath* local_file_path);
 
   DISALLOW_COPY_AND_ASSIGN(NativeMediaFileUtil);
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_MEDIA_NATIVE_MEDIA_FILE_UTIL_H_