Rename "CheckWritable" to "PrepareForWritableApp" to reflect the actual behavior.

chrome/browser/chromeos/file_manager/filesystem_api_util.h

Index: chrome/browser/chromeos/file_manager/filesystem_api_util.h
diff --git a/chrome/browser/chromeos/file_manager/filesystem_api_util.h b/chrome/browser/chromeos/file_manager/filesystem_api_util.h
index 3ce7a6be15c05f482090b3af7a99f835762bbbf6..307ab45bfee4308698d1ad3ef0216e4cff5b0dcb 100644
--- a/chrome/browser/chromeos/file_manager/filesystem_api_util.h
+++ b/chrome/browser/chromeos/file_manager/filesystem_api_util.h
@@ -40,9 +40,10 @@ void IsNonNativeLocalPathDirectory(
     const base::FilePath& path,
     const base::Callback<void(bool)>& callback);
 
-// Prepares a writable file at |path|, i.e., creates a file there if it didn't
-// exist, and asynchronously sends to |callback| whether it succeeded.
-void PrepareNonNativeLocalPathWritableFile(
+// Ensures a file to exist at |path|, i.e., it does nothing if a file is already

[benwells, 2014/06/02 23:30:31]

Nit: "Ensures a file exists at |path|..."

[kinaba, 2014/06/03 02:29:54]

Done.

+// present, or creates a file there if it isn't, and asynchronously sends to
+// |callback| whether it succeeded.
+void PrepareNonNativeLocalPathFileForWritableApp(

[benwells, 2014/06/02 23:30:31]

PathFile is weird. Maybe PrepareNonNativeLocalFileForWritableApp.

[kinaba, 2014/06/03 02:29:54]

Done.

     Profile* profile,
     const base::FilePath& path,
     const base::Callback<void(bool)>& callback);