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

Side by Side Diff: webkit/chromeos/fileapi/cros_mount_point_provider.cc

Issue 10828043: Wire up the deleteFileSystem operation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Modified CrosMountPointProvider too. Created 8 years, 4 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/chromeos/fileapi/cros_mount_point_provider.h" 5 #include "webkit/chromeos/fileapi/cros_mount_point_provider.h"
6 6
7 #include "base/chromeos/chromeos_version.h" 7 #include "base/chromeos/chromeos_version.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/message_loop.h" 10 #include "base/message_loop.h"
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 // TODO(zelidrag): Share this code with SandboxMountPointProvider impl. 131 // TODO(zelidrag): Share this code with SandboxMountPointProvider impl.
132 bool CrosMountPointProvider::IsRestrictedFileName(const FilePath& path) const { 132 bool CrosMountPointProvider::IsRestrictedFileName(const FilePath& path) const {
133 return false; 133 return false;
134 } 134 }
135 135
136 fileapi::FileSystemQuotaUtil* CrosMountPointProvider::GetQuotaUtil() { 136 fileapi::FileSystemQuotaUtil* CrosMountPointProvider::GetQuotaUtil() {
137 // No quota support. 137 // No quota support.
138 return NULL; 138 return NULL;
139 } 139 }
140 140
141 void CrosMountPointProvider::DeleteFileSystem(
142 const GURL& origin_url,
143 fileapi::FileSystemType type,
144 fileapi::FileSystemContext* context,
145 const DeleteFileSystemCallback& callback) const {
146 NOTREACHED();
tzik 2012/07/28 00:36:46 Could you invoke callback even after we reached NO
nhiroki (google) 2012/07/28 02:01:56 Done.
147 }
148
141 bool CrosMountPointProvider::HasMountPoint(const FilePath& mount_point) { 149 bool CrosMountPointProvider::HasMountPoint(const FilePath& mount_point) {
142 base::AutoLock locker(mount_point_map_lock_); 150 base::AutoLock locker(mount_point_map_lock_);
143 MountPointMap::const_iterator iter = mount_point_map_.find( 151 MountPointMap::const_iterator iter = mount_point_map_.find(
144 mount_point.BaseName().value()); 152 mount_point.BaseName().value());
145 DCHECK(iter == mount_point_map_.end() || 153 DCHECK(iter == mount_point_map_.end() ||
146 iter->second.local_root_path == mount_point.DirName()); 154 iter->second.local_root_path == mount_point.DirName());
147 return iter != mount_point_map_.end(); 155 return iter != mount_point_map_.end();
148 } 156 }
149 157
150 void CrosMountPointProvider::AddLocalMountPoint(const FilePath& mount_point) { 158 void CrosMountPointProvider::AddLocalMountPoint(const FilePath& mount_point) {
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
299 if (mount_prefix == filesystem_path) { 307 if (mount_prefix == filesystem_path) {
300 return true; 308 return true;
301 } else if (mount_prefix.AppendRelativePath(filesystem_path, virtual_path)) { 309 } else if (mount_prefix.AppendRelativePath(filesystem_path, virtual_path)) {
302 return true; 310 return true;
303 } 311 }
304 } 312 }
305 return false; 313 return false;
306 } 314 }
307 315
308 } // namespace chromeos 316 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698