Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(500)

Side by Side Diff: webkit/fileapi/isolated_mount_point_provider.cc

Issue 10914188: Added PlatformFileError param to FileSystemMountPointProvider::CreateFileSystemOperation function. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "webkit/fileapi/isolated_mount_point_provider.h" 5 #include "webkit/fileapi/isolated_mount_point_provider.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/file_path.h" 10 #include "base/file_path.h"
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 } 119 }
120 120
121 FilePath IsolatedMountPointProvider::GetPathForPermissionsCheck( 121 FilePath IsolatedMountPointProvider::GetPathForPermissionsCheck(
122 const FilePath& virtual_path) const { 122 const FilePath& virtual_path) const {
123 // For isolated filesystems we only check per-filesystem permissions. 123 // For isolated filesystems we only check per-filesystem permissions.
124 return FilePath(); 124 return FilePath();
125 } 125 }
126 126
127 FileSystemOperation* IsolatedMountPointProvider::CreateFileSystemOperation( 127 FileSystemOperation* IsolatedMountPointProvider::CreateFileSystemOperation(
128 const FileSystemURL& url, 128 const FileSystemURL& url,
129 FileSystemContext* context) const { 129 FileSystemContext* context,
130 base::PlatformFileError* error_code) const {
130 scoped_ptr<FileSystemOperationContext> operation_context( 131 scoped_ptr<FileSystemOperationContext> operation_context(
131 new FileSystemOperationContext(context)); 132 new FileSystemOperationContext(context));
132 if (url.type() == kFileSystemTypeNativeMedia || 133 if (url.type() == kFileSystemTypeNativeMedia ||
133 url.type() == kFileSystemTypeDeviceMedia) { 134 url.type() == kFileSystemTypeDeviceMedia) {
134 operation_context->set_media_path_filter(media_path_filter_.get()); 135 operation_context->set_media_path_filter(media_path_filter_.get());
135 operation_context->set_task_runner( 136 operation_context->set_task_runner(
136 context->task_runners()->media_task_runner()); 137 context->task_runners()->media_task_runner());
137 } 138 }
138 139
139 #if defined(SUPPORT_MEDIA_FILESYSTEM) 140 #if defined(SUPPORT_MEDIA_FILESYSTEM)
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
171 void IsolatedMountPointProvider::DeleteFileSystem( 172 void IsolatedMountPointProvider::DeleteFileSystem(
172 const GURL& origin_url, 173 const GURL& origin_url,
173 FileSystemType type, 174 FileSystemType type,
174 FileSystemContext* context, 175 FileSystemContext* context,
175 const DeleteFileSystemCallback& callback) { 176 const DeleteFileSystemCallback& callback) {
176 NOTREACHED(); 177 NOTREACHED();
177 callback.Run(base::PLATFORM_FILE_ERROR_INVALID_OPERATION); 178 callback.Run(base::PLATFORM_FILE_ERROR_INVALID_OPERATION);
178 } 179 }
179 180
180 } // namespace fileapi 181 } // namespace fileapi
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698