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

Unified Diff: chrome/browser/chromeos/extensions/file_manager/file_watcher.cc

Issue 23945002: file_manager: Move non-binding code to c/b/chromeos/file_manager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/extensions/file_manager/file_watcher.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/file_watcher.cc b/chrome/browser/chromeos/extensions/file_manager/file_watcher.cc
deleted file mode 100644
index 6f72071978c2368d93eccf31589ad199cfba97a1..0000000000000000000000000000000000000000
--- a/chrome/browser/chromeos/extensions/file_manager/file_watcher.cc
+++ /dev/null
@@ -1,116 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/chromeos/extensions/file_manager/file_watcher.h"
-
-#include "base/bind.h"
-#include "base/message_loop/message_loop_proxy.h"
-#include "chrome/browser/google_apis/task_util.h"
-#include "content/public/browser/browser_thread.h"
-
-using content::BrowserThread;
-
-namespace file_manager {
-namespace {
-
-// Creates a base::FilePathWatcher and starts watching at |watch_path| with
-// |callback|. Returns NULL on failure.
-base::FilePathWatcher* CreateAndStartFilePathWatcher(
- const base::FilePath& watch_path,
- const base::FilePathWatcher::Callback& callback) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
- DCHECK(!callback.is_null());
-
- scoped_ptr<base::FilePathWatcher> watcher(new base::FilePathWatcher);
- if (!watcher->Watch(watch_path, false /* recursive */, callback))
- return NULL;
-
- return watcher.release();
-}
-
-} // namespace
-
-FileWatcher::FileWatcher(const base::FilePath& virtual_path)
- : local_file_watcher_(NULL),
- virtual_path_(virtual_path),
- weak_ptr_factory_(this) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-}
-
-FileWatcher::~FileWatcher() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-
- BrowserThread::DeleteSoon(BrowserThread::FILE,
- FROM_HERE,
- local_file_watcher_);
-}
-
-void FileWatcher::AddExtension(const std::string& extension_id) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-
- extensions_[extension_id]++;
-}
-
-void FileWatcher::RemoveExtension(const std::string& extension_id) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-
- ExtensionCountMap::iterator it = extensions_.find(extension_id);
- if (it == extensions_.end()) {
- LOG(ERROR) << " Extension [" << extension_id
- << "] tries to unsubscribe from folder ["
- << virtual_path_.value()
- << "] it isn't subscribed";
- return;
- }
-
- // If entry found - decrease it's count and remove if necessary
- --it->second;
- if (it->second == 0)
- extensions_.erase(it);
-}
-
-std::vector<std::string> FileWatcher::GetExtensionIds() const {
- std::vector<std::string> extension_ids;
- for (ExtensionCountMap::const_iterator iter = extensions_.begin();
- iter != extensions_.end(); ++iter) {
- extension_ids.push_back(iter->first);
- }
- return extension_ids;
-}
-
-void FileWatcher::WatchLocalFile(
- const base::FilePath& local_path,
- const base::FilePathWatcher::Callback& file_watcher_callback,
- const BoolCallback& callback) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(!callback.is_null());
- DCHECK(!local_file_watcher_);
-
- BrowserThread::PostTaskAndReplyWithResult(
- BrowserThread::FILE,
- FROM_HERE,
- base::Bind(&CreateAndStartFilePathWatcher,
- local_path,
- google_apis::CreateRelayCallback(file_watcher_callback)),
- base::Bind(&FileWatcher::OnWatcherStarted,
- weak_ptr_factory_.GetWeakPtr(),
- callback));
-}
-
-void FileWatcher::OnWatcherStarted(
- const BoolCallback& callback,
- base::FilePathWatcher* file_watcher) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(!callback.is_null());
- DCHECK(!local_file_watcher_);
-
- if (file_watcher) {
- local_file_watcher_ = file_watcher;
- callback.Run(true);
- } else {
- callback.Run(false);
- }
-}
-
-} // namespace file_manager

Powered by Google App Engine
This is Rietveld 408576698