From 22dea52cdc3fd2eac4781f1eb2993550ea2b0dba Mon Sep 17 00:00:00 2001 From: "kinuko@chromium.org" Date: Wed, 29 May 2013 07:44:40 +0000 Subject: Rename FileSystemMountPointProvider::ValidateFileSystemRoot to OpenFileSystem This patch does: - Rename ValidateFileSystemRoot() to OpenFileSystem() to make it clearer what the method does (as the method is called in response to OpenFileSystem request from the renderer) - Also change boolean 'create' parameter to an enum (OpenFileSystemMode) BUG=243216 R=benjhayden@chromium.org, kinaba@chromium.org, tzik@chromium.org Review URL: https://codereview.chromium.org/16043006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@202801 0039d316-1c4b-4281-b951-d872f2087c98 --- webkit/browser/chromeos/fileapi/cros_mount_point_provider.cc | 6 +++--- webkit/browser/chromeos/fileapi/cros_mount_point_provider.h | 8 ++++---- 2 files changed, 7 insertions(+), 7 deletions(-) (limited to 'webkit/browser/chromeos') diff --git a/webkit/browser/chromeos/fileapi/cros_mount_point_provider.cc b/webkit/browser/chromeos/fileapi/cros_mount_point_provider.cc index 5f36f6e..aa279d2 100644 --- a/webkit/browser/chromeos/fileapi/cros_mount_point_provider.cc +++ b/webkit/browser/chromeos/fileapi/cros_mount_point_provider.cc @@ -90,11 +90,11 @@ bool CrosMountPointProvider::CanHandleType(fileapi::FileSystemType type) const { } } -void CrosMountPointProvider::ValidateFileSystemRoot( +void CrosMountPointProvider::OpenFileSystem( const GURL& origin_url, fileapi::FileSystemType type, - bool create, - const ValidateFileSystemCallback& callback) { + fileapi::OpenFileSystemMode mode, + const OpenFileSystemCallback& callback) { DCHECK(fileapi::IsolatedContext::IsIsolatedType(type)); // Nothing to validate for external filesystem. callback.Run(base::PLATFORM_FILE_OK); diff --git a/webkit/browser/chromeos/fileapi/cros_mount_point_provider.h b/webkit/browser/chromeos/fileapi/cros_mount_point_provider.h index 09ac76d..48cba9c 100644 --- a/webkit/browser/chromeos/fileapi/cros_mount_point_provider.h +++ b/webkit/browser/chromeos/fileapi/cros_mount_point_provider.h @@ -35,7 +35,7 @@ class FileAccessPermissions; class WEBKIT_STORAGE_EXPORT CrosMountPointProvider : public fileapi::ExternalFileSystemMountPointProvider { public: - using fileapi::FileSystemMountPointProvider::ValidateFileSystemCallback; + using fileapi::FileSystemMountPointProvider::OpenFileSystemCallback; using fileapi::FileSystemMountPointProvider::DeleteFileSystemCallback; // CrosMountPointProvider will take an ownership of a |mount_points| @@ -54,11 +54,11 @@ class WEBKIT_STORAGE_EXPORT CrosMountPointProvider // fileapi::FileSystemMountPointProvider overrides. virtual bool CanHandleType(fileapi::FileSystemType type) const OVERRIDE; - virtual void ValidateFileSystemRoot( + virtual void OpenFileSystem( const GURL& origin_url, fileapi::FileSystemType type, - bool create, - const ValidateFileSystemCallback& callback) OVERRIDE; + fileapi::OpenFileSystemMode mode, + const OpenFileSystemCallback& callback) OVERRIDE; virtual fileapi::FileSystemFileUtil* GetFileUtil( fileapi::FileSystemType type) OVERRIDE; virtual fileapi::AsyncFileUtil* GetAsyncFileUtil( -- cgit v1.1