Validate image files before writing them to media galleries.

chrome/browser/media_galleries/fileapi/media_file_validator_factory.h

+// Copyright 2013 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 CHROME_BROWSER_MEDIA_GALLERIES_FILEAPI_MEDIA_FILE_VALIDATOR_FACTORY_H_
+#define CHROME_BROWSER_MEDIA_GALLERIES_FILEAPI_MEDIA_FILE_VALIDATOR_FACTORY_H_
+
+#include "base/basictypes.h"
+#include "base/files/file_path.h"
+#include "webkit/fileapi/copy_or_move_file_validator.h"
+
+namespace fileapi {
+class FileSystemURL;
+}
+
+namespace chrome {
+

[Greg Billock, 2013/05/22 16:21:59]

class comment

[vandebo (ex-Chrome), 2013/05/22 18:41:17]

Done.

+class MediaFileValidatorFactory
+    : public fileapi::CopyOrMoveFileValidatorFactory {
+ public:
+  MediaFileValidatorFactory();
+  virtual ~MediaFileValidatorFactory();
+
+  virtual fileapi::CopyOrMoveFileValidator* CreateCopyOrMoveFileValidator(
+            const fileapi::FileSystemURL& src,

[Greg Billock, 2013/05/22 16:21:59]

Why does the factory method take these args? They seem unexpected for something
that will validate either a particular file, or files in general.

[vandebo (ex-Chrome), 2013/05/22 18:41:17]

There is a comment explaining the args in the parent class.  Added a comment to
indicate that this is an override.

+            const base::FilePath& platform_path) OVERRIDE;
+ private:
+  DISALLOW_COPY_AND_ASSIGN(MediaFileValidatorFactory);
+};
+
+}  // namespace chrome
+
+#endif  // CHROME_BROWSER_MEDIA_GALLERIES_FILEAPI_MEDIA_FILE_VALIDATOR_FACTORY_H_