Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(164)

Issue 3743004: Rename FileUtilProxy::Create to EnsureFileExists (Closed)

Created:
10 years, 2 months ago by kinuko
Modified:
9 years, 7 months ago
Reviewers:
dumi, brettw
CC:
chromium-reviews, darin-cc_chromium.org, pam+watch_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Rename FileUtilProxy::Create to EnsureFileExists per follow-up discussion on http://codereview.chromium.org/3717001/show Rename FileUtilProxy::Create to EnsureFileExists to make it clear that it doesn't open the file (or doesn't leave a file handle opened). Also fixes CreatePlatformFile to set |created| correctly when PLATFORM_FILE_CREATE flag is given (I think this is the correct behavior). BUG=none TEST=FileSystemOperationTest.* Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=62683

Patch Set 1 : '' #

Total comments: 1

Patch Set 2 : fixed comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+103 lines, -59 lines) Patch
M base/file_util_proxy.h View 1 3 chunks +19 lines, -7 lines 0 comments Download
M base/file_util_proxy.cc View 7 chunks +61 lines, -23 lines 0 comments Download
M base/platform_file_posix.cc View 1 chunk +2 lines, -1 line 0 comments Download
M base/platform_file_win.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/fileapi/file_system_operation.h View 1 chunk +6 lines, -6 lines 0 comments Download
M webkit/fileapi/file_system_operation.cc View 2 chunks +13 lines, -20 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
kinuko
10 years, 2 months ago (2010-10-14 02:23:20 UTC) #1
brettw
LGTM, thanks for the follow-up. http://codereview.chromium.org/3743004/diff/2001/3002 File base/file_util_proxy.h (right): http://codereview.chromium.org/3743004/diff/2001/3002#newcode70 base/file_util_proxy.h:70: // of the callback ...
10 years, 2 months ago (2010-10-14 16:29:26 UTC) #2
kinuko
10 years, 2 months ago (2010-10-15 00:14:19 UTC) #3
On 2010/10/14 16:29:26, brettw wrote:
> LGTM, thanks for the follow-up.
> 
> http://codereview.chromium.org/3743004/diff/2001/3002
> File base/file_util_proxy.h (right):
> 
> http://codereview.chromium.org/3743004/diff/2001/3002#newcode70
> base/file_util_proxy.h:70: // of the callback is set true in the case.
> Can you explicitly mention that that created flag will be false if the file
> already exists, and what also happens if the file doesn't exist but couldn't
be
> created (like maybe the directory is invalid).

Fixed the comment to explicitly mention each case.  Will be submitting...

Powered by Google App Engine
This is Rietveld 408576698