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

Unified Diff: content/public/test/test_file_system_backend.cc

Issue 452043003: [ew] Add basic classes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments. Created 6 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 side-by-side diff with in-line comments
Download patch
Index: content/public/test/test_file_system_backend.cc
diff --git a/content/public/test/test_file_system_backend.cc b/content/public/test/test_file_system_backend.cc
index aa14959cbb6e3885f0a6dae656b8a3bc05b31c0c..df3f71bac126263a43b3bbcda8e6d3c4bf10389c 100644
--- a/content/public/test/test_file_system_backend.cc
+++ b/content/public/test/test_file_system_backend.cc
@@ -9,7 +9,11 @@
#include <vector>
#include "base/file_util.h"
+#include "base/files/file.h"
+#include "base/memory/weak_ptr.h"
+#include "base/observer_list.h"
#include "base/sequenced_task_runner.h"
+#include "base/thread_task_runner_handle.h"
#include "webkit/browser/blob/file_stream_reader.h"
#include "webkit/browser/fileapi/copy_or_move_file_validator.h"
#include "webkit/browser/fileapi/file_observers.h"
@@ -20,6 +24,7 @@
#include "webkit/browser/fileapi/native_file_util.h"
#include "webkit/browser/fileapi/quota/quota_reservation.h"
#include "webkit/browser/fileapi/sandbox_file_stream_writer.h"
+#include "webkit/browser/fileapi/watcher_manager.h"
#include "webkit/browser/quota/quota_manager.h"
#include "webkit/common/fileapi/file_system_util.h"
@@ -32,6 +37,7 @@ namespace content {
namespace {
+// Stub implementation of fileapi::LocalFileUtil.
class TestFileUtil : public fileapi::LocalFileUtil {
public:
explicit TestFileUtil(const base::FilePath& base_path)
@@ -51,6 +57,101 @@ class TestFileUtil : public fileapi::LocalFileUtil {
base::FilePath base_path_;
};
+// Stub implementation of fileapi::WatcherManager. Emits a fake notification
+// after a directory watcher is set successfully.
+class TestWatcherManager : public fileapi::WatcherManager {
+ public:
+ TestWatcherManager() : weak_ptr_factory_(this) {}
+ virtual ~TestWatcherManager() {}
+
+ // fileapi::WatcherManager overrides.
+ virtual void AddObserver(Observer* observer) OVERRIDE {
+ observers_.AddObserver(observer);
+ }
+
+ virtual void RemoveObserver(Observer* observer) OVERRIDE {
+ observers_.RemoveObserver(observer);
+ }
+
+ virtual bool HasObserver(Observer* observer) const OVERRIDE {
+ return observers_.HasObserver(observer);
+ }
+
+ virtual void WatchDirectory(const fileapi::FileSystemURL& url,
+ bool recursive,
+ const StatusCallback& callback) OVERRIDE {
+ if (recursive) {
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE,
+ base::Bind(callback, base::File::FILE_ERROR_INVALID_OPERATION));
+ return;
+ }
+
+ const GURL gurl = url.ToGURL();
+ if (watched_urls_.find(gurl) != watched_urls_.end()) {
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE, base::Bind(callback, base::File::FILE_ERROR_EXISTS));
+ return;
+ }
+
+ watched_urls_.insert(gurl);
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE, base::Bind(callback, base::File::FILE_OK));
+
+ // Send a fake changed notification.
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE,
+ base::Bind(&TestWatcherManager::SendFakeChangeNotification,
+ weak_ptr_factory_.GetWeakPtr(),
+ url));
+
+ // Send a fake removed notification.
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE,
+ base::Bind(&TestWatcherManager::SendFakeRemoveNotification,
+ weak_ptr_factory_.GetWeakPtr(),
+ url));
+ }
+
+ virtual void UnwatchEntry(const fileapi::FileSystemURL& url,
+ const StatusCallback& callback) OVERRIDE {
+ const GURL gurl = url.ToGURL();
+ if (watched_urls_.find(gurl) == watched_urls_.end()) {
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE, base::Bind(callback, base::File::FILE_ERROR_NOT_FOUND));
+ return;
+ }
+
+ watched_urls_.erase(gurl);
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE, base::Bind(callback, base::File::FILE_OK));
+ }
+
+ private:
+ // Sends a fake notification to each observer about a changed entry
+ // represented by |url|, as long as it is still being watched.
+ void SendFakeChangeNotification(const fileapi::FileSystemURL& url) {
+ if (watched_urls_.find(url.ToGURL()) == watched_urls_.end())
+ return;
+
+ FOR_EACH_OBSERVER(Observer, observers_, OnEntryChanged(url));
+ }
+
+ // Sends a fake notification to each observer about a removed entry
+ // represented by |url|, as long as it is still being watched.
+ void SendFakeRemoveNotification(const fileapi::FileSystemURL& url) {
+ if (watched_urls_.find(url.ToGURL()) == watched_urls_.end())
+ return;
+
+ FOR_EACH_OBSERVER(Observer, observers_, OnEntryRemoved(url));
+ }
+
+ ObserverList<Observer> observers_;
+ std::set<GURL> watched_urls_;
+
+ base::WeakPtrFactory<TestWatcherManager> weak_ptr_factory_;
+};
+
} // namespace
// This only supports single origin.
@@ -163,6 +264,7 @@ TestFileSystemBackend::TestFileSystemBackend(
: base_path_(base_path),
file_util_(
new fileapi::AsyncFileUtilAdapter(new TestFileUtil(base_path))),
+ watcher_manager_(new TestWatcherManager()),
quota_util_(new QuotaUtil(task_runner)),
require_copy_or_move_validator_(false) {
}
@@ -190,6 +292,11 @@ fileapi::AsyncFileUtil* TestFileSystemBackend::GetAsyncFileUtil(
return file_util_.get();
}
+fileapi::WatcherManager* TestFileSystemBackend::GetWatcherManager(
+ fileapi::FileSystemType type) {
+ return watcher_manager_.get();
+}
+
fileapi::CopyOrMoveFileValidatorFactory*
TestFileSystemBackend::GetCopyOrMoveFileValidatorFactory(
fileapi::FileSystemType type,

Powered by Google App Engine
This is Rietveld 408576698