Make chrome.filesystem API fully work with the Chrome OS Google Drive folder.

chrome/browser/extensions/api/file_system/file_system_api.cc

 // 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.
 
 #include "chrome/browser/extensions/api/file_system/file_system_api.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
@@ skipping 21 matching lines @@
 #include "webkit/fileapi/external_mount_points.h"
 #include "webkit/fileapi/file_system_types.h"
 #include "webkit/fileapi/file_system_util.h"
 #include "webkit/fileapi/isolated_context.h"
 
 #if defined(OS_MACOSX)
 #include <CoreFoundation/CoreFoundation.h>
 #include "base/mac/foundation_util.h"
 #endif
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/drive/drive_file_system_util.h"
+#endif
+
 using fileapi::IsolatedContext;
 
 const char kInvalidParameters[] = "Invalid parameters";
 const char kSecurityError[] = "Security error";
 const char kInvalidCallingPage[] = "Invalid calling page. This function can't "
     "be called from a background page.";
 const char kUserCancelled[] = "User cancelled";
 const char kWritableFileError[] = "Invalid file for writing";
 const char kRequiresFileSystemWriteError[] =
     "Operation requires fileSystem.write permission";
@@ skipping 133 matching lines @@
                        base::PLATFORM_FILE_WRITE;
   base::PlatformFile file = base::CreatePlatformFile(path, creation_flags,
                                                      NULL, &error);
   // Close the file so we don't keep a lock open.
   if (file != base::kInvalidPlatformFileValue)
     base::ClosePlatformFile(file);
   return error == base::PLATFORM_FILE_OK ||
          error == base::PLATFORM_FILE_ERROR_EXISTS;
 }
 
+void CheckLocalWritableFile(const base::FilePath& path,
+                            const base::Closure& on_success,
+                            const base::Closure& on_failure) {
+  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
+  content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
+      DoCheckWritableFile(path) ? on_success : on_failure);
+}
+
+#if defined(OS_CHROMEOS)
+void CheckRemoteWritableFile(const base::Closure& on_success,
+                             const base::Closure& on_failure,
+                             drive::DriveFileError error,
+                             const base::FilePath& path) {
+  content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
+      error == drive::DRIVE_FILE_OK ? on_success : on_failure);
+}
+#endif
+
 // Expand the mime-types and extensions provided in an AcceptOption, returning
 // them within the passed extension vector. Returns false if no valid types
 // were found.
 bool GetFileTypesFromAcceptOption(
     const file_system::AcceptOption& accept_option,
     std::vector<base::FilePath::StringType>* extensions,
     string16* description) {
   std::set<base::FilePath::StringType> extension_set;
   int description_id = 0;
 
@@ skipping 72 matching lines @@
 
 bool FileSystemEntryFunction::HasFileSystemWritePermission() {
   const extensions::Extension* extension = GetExtension();
   if (!extension)
     return false;
 
   return extension->HasAPIPermission(APIPermission::kFileSystemWrite);
 }
 
 void FileSystemEntryFunction::CheckWritableFile(const base::FilePath& path) {
-  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
-  if (DoCheckWritableFile(path)) {
-    content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
-        base::Bind(&FileSystemEntryFunction::RegisterFileSystemAndSendResponse,
-            this, path, WRITABLE));
+  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+  base::Closure on_success =
+      base::Bind(&FileSystemEntryFunction::RegisterFileSystemAndSendResponse,
+                 this, path, WRITABLE);
+  base::Closure on_failure =
+      base::Bind(&FileSystemEntryFunction::HandleWritableFileError, this);
+
+#if defined(OS_CHROMEOS)
+  if (drive::util::IsUnderDriveMountPoint(path)) {
+    drive::util::PrepareWritableFileAndRun(profile_, path,
+        base::Bind(&CheckRemoteWritableFile, on_success, on_failure));
     return;
   }
-
-  content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
-      base::Bind(&FileSystemEntryFunction::HandleWritableFileError, this));
+#endif
+  content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
+      base::Bind(&CheckLocalWritableFile, path, on_success, on_failure));
 }
 
 void FileSystemEntryFunction::RegisterFileSystemAndSendResponse(
     const base::FilePath& path, EntryType entry_type) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   fileapi::IsolatedContext* isolated_context =
       fileapi::IsolatedContext::GetInstance();
   DCHECK(isolated_context);
 
@@ skipping 27 matching lines @@
   if (!HasFileSystemWritePermission()) {
     error_ = kRequiresFileSystemWriteError;
     return false;
   }
 
   base::FilePath path;
   if (!GetFilePathOfFileEntry(filesystem_name, filesystem_path,
                               render_view_host_, &path, &error_))
     return false;
 
-  content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
-      base::Bind(&FileSystemGetWritableEntryFunction::CheckWritableFile,
-          this, path));
+  CheckWritableFile(path);
   return true;
 }
 
 bool FileSystemIsWritableEntryFunction::RunImpl() {
   std::string filesystem_name;
   std::string filesystem_path;
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, &filesystem_name));
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(1, &filesystem_path));
 
   std::string filesystem_id;
@@ skipping 147 matching lines @@
   // For testing on Chrome OS, where to deal with remote and local paths
   // smoothly, all accessed paths need to be registered in the list of
   // external mount points.
   fileapi::ExternalMountPoints::GetSystemInstance()->RegisterFileSystem(
       name, fileapi::kFileSystemTypeNativeLocal, path);
 }
 
 void FileSystemChooseEntryFunction::FileSelected(const base::FilePath& path,
                                                 EntryType entry_type) {
   if (entry_type == WRITABLE) {
-    content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
-        base::Bind(&FileSystemChooseEntryFunction::CheckWritableFile,
-            this, path));
+    CheckWritableFile(path);
     return;
   }
 
   // Don't need to check the file, it's for reading.
   RegisterFileSystemAndSendResponse(path, READ_ONLY);
 }
 
 void FileSystemChooseEntryFunction::FileSelectionCanceled() {
   error_ = kUserCancelled;
   SendResponse(false);
@@ skipping 89 matching lines @@
 
     BuildFileTypeInfo(&file_type_info, suggested_extension,
         options->accepts.get(), options->accepts_all_types.get());
   }
 
   if (entry_type == WRITABLE && !HasFileSystemWritePermission()) {
     error_ = kRequiresFileSystemWriteError;
     return false;
   }
 
-  if (entry_type != WRITABLE)
-    file_type_info.support_drive = true;
-
+  file_type_info.support_drive = true;
   return ShowPicker(suggested_name, file_type_info, picker_type, entry_type);
 }
 
 }  // namespace extensions