Add the methods to change and get a posix file permission to file_util.

base/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.
 
 // This file contains utility functions for dealing with the local
 // filesystem.
 
 #ifndef BASE_FILE_UTIL_H_
 #define BASE_FILE_UTIL_H_
 #pragma once
@@ skipping 178 matching lines @@
 BASE_EXPORT bool ReadFromFD(int fd, char* buffer, size_t bytes);
 
 // Creates a symbolic link at |symlink| pointing to |target|.  Returns
 // false on failure.
 BASE_EXPORT bool CreateSymbolicLink(const FilePath& target,
                                     const FilePath& symlink);
 
 // Reads the given |symlink| and returns where it points to in |target|.
 // Returns false upon failure.
 BASE_EXPORT bool ReadSymbolicLink(const FilePath& symlink, FilePath* target);
+
+// Read the permission of the given |path|. |mode| is file permission bits.

[satorux1, 2012/07/03 06:16:55]

Read -> Reads

[satorux1, 2012/07/03 06:16:55]

Please also mention about how symbolic links are handled.

[yoshiki, 2012/07/04 07:57:57]

Done x2.

+BASE_EXPORT bool GetPosixFilePermissions(const FilePath& path,
+                                         mode_t* mode);

[satorux1, 2012/07/03 06:16:55]

Not sure if using mode_t is a good idea. we might want to define make it 'int'
and define easy-to-read enumerators like READ_BY_OTHERS than letting clients use
hard-to-read constants like  S_IROTH. 

[yoshiki, 2012/07/04 07:57:57]

Done.

+// Change the permission of the given |path|.

[satorux1, 2012/07/03 06:16:55]

Change -> Changes

[yoshiki, 2012/07/04 07:57:57]

Done.

+BASE_EXPORT bool ChangePosixFilePermissions(const FilePath& path,
+                                            mode_t mode_bits_to_set,
+                                            mode_t mode_bits_to_clear);

[satorux1, 2012/07/03 06:16:55]

I think taking two parameters for chaning the permissions is rather awkward.
Would be better to be compatible with chmod(2), that only takes one parameter.

[yoshiki, 2012/07/04 07:57:57]

Done. I added another method SetPosixFilePermissions().

 #endif  // defined(OS_POSIX)
 
 #if defined(OS_WIN)
 enum ShortcutOptions {
   SHORTCUT_NO_OPTIONS = 0,
   // Set DualMode property for Windows 8 Metro-enabled shortcuts.
   SHORTCUT_DUAL_MODE = 1 << 0,
   // Create a new shortcut (overwriting if necessary). If not specified, only
   // non-null properties on an existing shortcut will be modified.
   SHORTCUT_CREATE_ALWAYS = 1 << 1,
@@ skipping 427 matching lines @@
 };
 
 // Attempts determine the FileSystemType for |path|.
 // Returns false if |path| doesn't exist.
 BASE_EXPORT bool GetFileSystemType(const FilePath& path, FileSystemType* type);
 #endif
 
 }  // namespace file_util
 
 #endif  // BASE_FILE_UTIL_H_