diff options
author | kinaba@chromium.org <kinaba@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-30 15:01:40 +0000 |
---|---|---|
committer | kinaba@chromium.org <kinaba@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-30 15:01:40 +0000 |
commit | d9b7a5bfb1a381cce156e17ebe457604563f674b (patch) | |
tree | 32daddc5a1d0d7511d0fc85942d584bfc11aad5c /webkit | |
parent | 1895c6347ab70a0f0cc18ed0511949f0513159ee (diff) | |
download | chromium_src-d9b7a5bfb1a381cce156e17ebe457604563f674b.zip chromium_src-d9b7a5bfb1a381cce156e17ebe457604563f674b.tar.gz chromium_src-d9b7a5bfb1a381cce156e17ebe457604563f674b.tar.bz2 |
Allow permission setting errors during folder creation in file API.
BUG=chrome-os-partner:23551
Review URL: https://codereview.chromium.org/50333002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231831 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit')
-rw-r--r-- | webkit/browser/fileapi/native_file_util.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/webkit/browser/fileapi/native_file_util.cc b/webkit/browser/fileapi/native_file_util.cc index eec9670..df290c5 100644 --- a/webkit/browser/fileapi/native_file_util.cc +++ b/webkit/browser/fileapi/native_file_util.cc @@ -140,8 +140,12 @@ PlatformFileError NativeFileUtil::CreateDirectory( if (!file_util::CreateDirectory(path)) return base::PLATFORM_FILE_ERROR_FAILED; - if (!SetPlatformSpecificDirectoryPermissions(path)) - return base::PLATFORM_FILE_ERROR_FAILED; + if (!SetPlatformSpecificDirectoryPermissions(path)) { + // Since some file systems don't support permission setting, we do not treat + // an error from the function as the failure of copying. Just log it. + LOG(WARNING) << "Setting directory permission failed: " + << path.AsUTF8Unsafe(); + } return base::PLATFORM_FILE_OK; } |