Added new fileBrowserPrivate and fileHandler extension APIs

webkit/fileapi/sandbox_mount_point_provider.h

 // Copyright (c) 2011 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.
 
 #ifndef WEBKIT_FILEAPI_SANDBOX_MOUNT_POINT_PROVIDER_H_
 #define WEBKIT_FILEAPI_SANDBOX_MOUNT_POINT_PROVIDER_H_
 
 #include <string>
+#include <vector>
 
 #include "base/file_path.h"
 #include "googleurl/src/gurl.h"
 #include "webkit/fileapi/file_system_mount_point_provider.h"
 
 class GURL;
 
 namespace base {
 class MessageLoopProxy;
 }
 
 namespace fileapi {
 
 class SandboxMountPointProvider : public FileSystemMountPointProvider {
  public:
 
   SandboxMountPointProvider(
       FileSystemPathManager* path_manager,
       scoped_refptr<base::MessageLoopProxy> file_message_loop,
       const FilePath& profile_path);
   virtual ~SandboxMountPointProvider();
 
-  // Checks if mount point access is allowed from |origin_url|.
-  virtual bool IsAccessAllowed(const GURL& origin_url);
+  // Checks if access to |virtual_path| is allowed from |origin_url|.
+  virtual bool IsAccessAllowed(const GURL& origin_url,
+                               const FilePath& virtual_path);
+  virtual void GrantAccess(const GURL& origin_url,
+                           const FilePath& virtual_path) {}

[ericu, 2011/04/07 01:40:58]

Should this ever get called?  How about a NOTREACHED?

[zel, 2011/04/07 02:54:42]

Done.

 
   // Retrieves the root path for the given |origin_url| and |type|, and
   // calls the given |callback| with the root path and name.
   // If |create| is true this also creates the directory if it doesn't exist.
   virtual void GetFileSystemRootPath(
       const GURL& origin_url,
       FileSystemType type,
       bool create,
       FileSystemPathManager::GetRootPathCallback* callback);
 
@@ skipping 11 matching lines @@
   static const char kPersistentName[];
   static const char kTemporaryName[];
 
   const FilePath& base_path() const {
     return base_path_;
   }
 
   // Checks if a given |name| contains any restricted names/chars in it.
   virtual bool IsRestrictedFileName(const FilePath& filename) const;
 
+  virtual std::vector<FilePath> GetRootDirectories() const {
+    return  std::vector<FilePath>();

[ericu, 2011/04/07 01:40:58]

Same here.  Perhaps these shouldn't really live on the superclass, but should
instead be on your specific subclass?

[zel, 2011/04/07 02:54:42]

I've added NOTREACHED() here. Done.

+  }
+
   // Returns the origin identifier string, which is used as a part of the
   // sandboxed path component, for the given |url|.
   static std::string GetOriginIdentifierFromURL(const GURL& url);
 
   // Gets a base directory path of the sandboxed filesystem that is
   // specified by |origin_identifier| and |type|.
   // |base_path| must be pointing the FileSystem's data directory
   // under the profile directory, i.e. <profile_dir>/kFileSystemDirectory.
   // Returns an empty path if any of the given parameters are invalid.
   // Returned directory path does not contain 'unique' part, therefore
@@ skipping 39 matching lines @@
 
   const FilePath base_path_;
 
   DISALLOW_COPY_AND_ASSIGN(SandboxMountPointProvider);
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_SANDBOX_MOUNT_POINT_PROVIDER_H_