Integrate external mount points to IsolatedContext

content/browser/fileapi/fileapi_message_filter.cc

Index: content/browser/fileapi/fileapi_message_filter.cc
diff --git a/content/browser/fileapi/fileapi_message_filter.cc b/content/browser/fileapi/fileapi_message_filter.cc
index da6581b5d4d5a24abe3222a91f9a8374afa1a445..2439ad1ec75fd5d84cd8b382242dfb3856ca7391 100644
--- a/content/browser/fileapi/fileapi_message_filter.cc
+++ b/content/browser/fileapi/fileapi_message_filter.cc
@@ -770,11 +770,9 @@ bool FileAPIMessageFilter::HasPermissionsForFile(
 
     // Access permission to the file system overrides the file permission
     // (if and only if they accessed via an isolated file system).
-    bool success = policy->HasPermissionsForFileSystem(
-        process_id_, url.filesystem_id(), permissions);
-    if (!success)
-      *error = base::PLATFORM_FILE_ERROR_SECURITY;
-    return success;
+    if (policy->HasPermissionsForFileSystem(process_id_, url.filesystem_id(),
+                                            permissions))
+      return true;

[kmadhusu, 2012/08/14 18:01:13]

Don't we need to set *error if HasPermissionsForFileSystem call failed? Is there
any situation when there is no permission to access the file system but there is
permission to read the file? 

[kinuko, 2012/08/20 08:54:35]

I needed this hack for cros's external mount points. I added
GrantReadWriteFileSystem for them in file_browser_private_api to revert this
change.

   }
 
   file_path = mount_point_provider->GetPathForPermissionsCheck(url.path());