Moving Entry from base::file_util_proxy namespace to within class base::FileU...

base/file_util_proxy.h

 // Copyright (c) 2010 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 BASE_FILE_UTIL_PROXY_H_
 #define BASE_FILE_UTIL_PROXY_H_
 
 #include <vector>
 
 #include "base/callback.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/platform_file.h"
 #include "base/ref_counted.h"
 #include "base/tracked_objects.h"
 
 namespace base {
 
-namespace file_util_proxy {
-
-// Holds metadata for file or directory entry.
-struct Entry {
-  FilePath::StringType name;
-  bool is_directory;
-};
-
-}  // namespace file_util_proxy
-
 class MessageLoopProxy;
 class Time;
 
 // This class provides asynchronous access to common file routines.
 class FileUtilProxy {
  public:
+  // Holds metadata for file or directory entry.

[kinuko, 2010/11/02 03:31:53]

Would be nice to mention that it's used by ReadDirectoryCallback.

+  struct Entry {
+    FilePath::StringType name;
+    bool is_directory;
+  };
+
   // This callback is used by methods that report only an error code.  It is
   // valid to pass NULL as the callback parameter to any function that takes a
   // StatusCallback, in which case the operation will complete silently.
-  typedef Callback1<base::PlatformFileError /* error code */
-                    >::Type StatusCallback;
+  typedef Callback1<PlatformFileError /* error code */>::Type StatusCallback;
 
   // Creates or opens a file with the given flags.  It is invalid to pass NULL
   // for the callback.
   // If PLATFORM_FILE_CREATE is set in |file_flags| it always tries to create
   // a new file at the given |file_path| and calls back with
   // PLATFORM_FILE_ERROR_FILE_EXISTS if the |file_path| already exists.
-  typedef Callback3<base::PlatformFileError /* error code */,
-                    base::PassPlatformFile,
+  typedef Callback3<PlatformFileError /* error code */,
+                    PassPlatformFile,
                     bool /* created */>::Type CreateOrOpenCallback;
   static bool CreateOrOpen(scoped_refptr<MessageLoopProxy> message_loop_proxy,
                            const FilePath& file_path,
                            int file_flags,
                            CreateOrOpenCallback* callback);
 
   // Creates a temporary file for writing.  The path and an open file handle
   // are returned.  It is invalid to pass NULL for the callback.
-  typedef Callback3<base::PlatformFileError /* error code */,
-                    base::PassPlatformFile,
+  typedef Callback3<PlatformFileError /* error code */,
+                    PassPlatformFile,
                     FilePath>::Type CreateTemporaryCallback;
   static bool CreateTemporary(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
       CreateTemporaryCallback* callback);
 
   // Close the given file handle.
   static bool Close(scoped_refptr<MessageLoopProxy> message_loop_proxy,
-                    base::PlatformFile,
+                    PlatformFile,
                     StatusCallback* callback);
 
   // Ensures that the given |file_path| exist.  This creates a empty new file
   // at |file_path| if the |file_path| does not exist.
   // If a new file han not existed and is created at the |file_path|,
   // |created| of the callback argument is set true and |error code|
   // is set PLATFORM_FILE_OK.
   // If the file already exists, |created| is set false and |error code|
   // is set PLATFORM_FILE_OK.
   // If the file hasn't existed but it couldn't be created for some other
   // reasons, |created| is set false and |error code| indicates the error.
-  typedef Callback2<base::PlatformFileError /* error code */,
+  typedef Callback2<PlatformFileError /* error code */,
                     bool /* created */>::Type EnsureFileExistsCallback;
   static bool EnsureFileExists(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
       const FilePath& file_path,
       EnsureFileExistsCallback* callback);
 
   // Retrieves the information about a file. It is invalid to pass NULL for the
   // callback.
-  typedef Callback2<base::PlatformFileError /* error code */,
-                    const base::PlatformFileInfo& /* file_info */
+  typedef Callback2<PlatformFileError /* error code */,
+                    const PlatformFileInfo& /* file_info */
                     >::Type GetFileInfoCallback;
   static bool GetFileInfo(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
       const FilePath& file_path,
       GetFileInfoCallback* callback);
 
   static bool GetFileInfoFromPlatformFile(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
-      base::PlatformFile file,
+      PlatformFile file,
       GetFileInfoCallback* callback);
 
-  typedef Callback2<base::PlatformFileError /* error code */,
-      const std::vector<base::file_util_proxy::Entry>&
-       >::Type ReadDirectoryCallback;
+  typedef Callback2<PlatformFileError /* error code */,
+      const std::vector<Entry>&>::Type ReadDirectoryCallback;

[kinuko, 2010/11/02 03:31:53]

nit: indent

   static bool ReadDirectory(scoped_refptr<MessageLoopProxy> message_loop_proxy,
                             const FilePath& file_path,
                             ReadDirectoryCallback* callback);
 
   // Copies a file or a directory from |src_file_path| to |dest_file_path|
   // Error cases:
   // If destination file doesn't exist or destination's parent
   // doesn't exists.
   // If source dir exists but destination path is an existing file.
   // If source file exists but destination path is an existing directory.
@@ skipping 29 matching lines @@
       StatusCallback* callback);
 
   // Deletes a directory and all of its contents.
   static bool RecursiveDelete(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
       const FilePath& file_path,
       StatusCallback* callback);
 
   // Reads from a file. On success, the file pointer is moved to position
   // |offset + bytes_to_read| in the file. The callback can be NULL.
-  typedef Callback2<base::PlatformFileError /* error code */,
+  typedef Callback2<PlatformFileError /* error code */,
                     int /* bytes read/written */>::Type ReadWriteCallback;
   static bool Read(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
-      base::PlatformFile file,
+      PlatformFile file,
       int64 offset,
       char* buffer,
       int bytes_to_read,
       ReadWriteCallback* callback);
 
   // Writes to a file. If |offset| is greater than the length of the file,
   // |false| is returned. On success, the file pointer is moved to position
   // |offset + bytes_to_write| in the file. The callback can be NULL.
   static bool Write(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
-      base::PlatformFile file,
+      PlatformFile file,
       int64 offset,
       const char* buffer,
       int bytes_to_write,
       ReadWriteCallback* callback);
 
   // Touches a file. The callback can be NULL.
   static bool Touch(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
-      base::PlatformFile file,
-      const base::Time& last_access_time,
-      const base::Time& last_modified_time,
+      PlatformFile file,
+      const Time& last_access_time,
+      const Time& last_modified_time,
       StatusCallback* callback);
 
   // Touches a file. The callback can be NULL.
   static bool Touch(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
       const FilePath& file_path,
-      const base::Time& last_access_time,
-      const base::Time& last_modified_time,
+      const Time& last_access_time,
+      const Time& last_modified_time,
       StatusCallback* callback);
 
   // Truncates a file to the given length. If |length| is greater than the
   // current length of the file, the file will be extended with zeroes.
   // The callback can be NULL.
   static bool Truncate(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
-      base::PlatformFile file,
+      PlatformFile file,
       int64 length,
       StatusCallback* callback);
 
   // Truncates a file to the given length. If |length| is greater than the
   // current length of the file, the file will be extended with zeroes.
   // The callback can be NULL.
   static bool Truncate(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
       const FilePath& path,
       int64 length,
       StatusCallback* callback);
 
   // Flushes a file. The callback can be NULL.
   static bool Flush(
       scoped_refptr<MessageLoopProxy> message_loop_proxy,
-      base::PlatformFile file,
+      PlatformFile file,
       StatusCallback* callback);
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(FileUtilProxy);
 };
 
 }  // namespace base
 
 #endif  // BASE_FILE_UTIL_PROXY_H_