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

Unified Diff: webkit/browser/fileapi/file_system_context.cc

Issue 18668003: SyncFS: Introduce SyncFileSystemBackend (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review fix Created 7 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: webkit/browser/fileapi/file_system_context.cc
diff --git a/webkit/browser/fileapi/file_system_context.cc b/webkit/browser/fileapi/file_system_context.cc
index 50ba578fd220db68e72a495e31447e3dbe3502de..2ef580bbcbdd4c41feeced77ec5249ec4c2febd4 100644
--- a/webkit/browser/fileapi/file_system_context.cc
+++ b/webkit/browser/fileapi/file_system_context.cc
@@ -228,25 +228,17 @@ bool FileSystemContext::IsSandboxFileSystem(FileSystemType type) const {
const UpdateObserverList* FileSystemContext::GetUpdateObservers(
FileSystemType type) const {
- // Currently update observer is only available in SandboxFileSystemBackend
- // and TestFileSystemBackend.
- // TODO(kinuko): Probably GetUpdateObservers() virtual method should be
- // added to FileSystemBackend interface and be called like
- // other GetFoo() methods do.
- if (sandbox_backend_->CanHandleType(type))
- return sandbox_backend_->GetUpdateObservers(type);
- if (type != kFileSystemTypeTest)
- return NULL;
FileSystemBackend* backend = GetFileSystemBackend(type);
- return static_cast<TestFileSystemBackend*>(
- backend)->GetUpdateObservers(type);
+ if (backend->GetQuotaUtil())
+ return backend->GetQuotaUtil()->GetUpdateObservers(type);
+ return NULL;
}
const AccessObserverList* FileSystemContext::GetAccessObservers(
FileSystemType type) const {
- // Currently access observer is only available in SandboxFileSystemBackend.
- if (sandbox_backend_->CanHandleType(type))
- return sandbox_backend_->GetAccessObservers(type);
+ FileSystemBackend* backend = GetFileSystemBackend(type);
+ if (backend->GetQuotaUtil())
+ return backend->GetQuotaUtil()->GetAccessObservers(type);
return NULL;
}
@@ -334,21 +326,26 @@ scoped_ptr<FileStreamWriter> FileSystemContext::CreateFileStreamWriter(
return backend->CreateFileStreamWriter(url, offset, this);
}
+// TODO(nhiroki): Move into SyncFileSystemBackend (http://crbug.com/242422).
void FileSystemContext::SetLocalFileChangeTracker(
scoped_ptr<sync_file_system::LocalFileChangeTracker> tracker) {
DCHECK(!change_tracker_.get());
DCHECK(tracker.get());
change_tracker_ = tracker.Pass();
- sandbox_backend_->AddFileUpdateObserver(
+
+ FileSystemBackend* backend = GetFileSystemBackend(kFileSystemTypeSyncable);
+ DCHECK(backend->GetQuotaUtil());
+ backend->GetQuotaUtil()->AddFileUpdateObserver(
kFileSystemTypeSyncable,
change_tracker_.get(),
task_runners_->file_task_runner());
- sandbox_backend_->AddFileChangeObserver(
+ backend->GetQuotaUtil()->AddFileChangeObserver(
kFileSystemTypeSyncable,
change_tracker_.get(),
task_runners_->file_task_runner());
}
+// TODO(nhiroki): Move into SyncFileSystemBackend (http://crbug.com/242422).
void FileSystemContext::set_sync_context(
sync_file_system::LocalFileSyncContext* sync_context) {
sync_context_ = sync_context;

Powered by Google App Engine
This is Rietveld 408576698