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

Unified Diff: components/password_manager/core/browser/password_store.cc

Issue 246253014: Remove friend classes from PasswordStore. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nits Created 6 years, 8 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: components/password_manager/core/browser/password_store.cc
diff --git a/components/password_manager/core/browser/password_store.cc b/components/password_manager/core/browser/password_store.cc
index b23b3ec9d54f8736b82bab0caace223630368f5a..0dd5684d6cb68bad96506eb2f8738ea21ae44c9a 100644
--- a/components/password_manager/core/browser/password_store.cc
+++ b/components/password_manager/core/browser/password_store.cc
@@ -163,6 +163,14 @@ void PasswordStore::RemoveObserver(Observer* observer) {
observers_->RemoveObserver(observer);
}
+bool PasswordStore::ScheduleTask(const base::Closure& task) {
+ scoped_refptr<base::SingleThreadTaskRunner> task_runner(
+ GetBackgroundTaskRunner());
+ if (task_runner.get())
+ return task_runner->PostTask(FROM_HERE, task);
+ return false;
+}
+
void PasswordStore::Shutdown() {
#if defined(PASSWORD_MANAGER_ENABLE_SYNC)
ScheduleTask(base::Bind(&PasswordStore::DestroySyncableService, this));
@@ -181,14 +189,6 @@ base::WeakPtr<syncer::SyncableService>
PasswordStore::~PasswordStore() { DCHECK(shutdown_called_); }
-bool PasswordStore::ScheduleTask(const base::Closure& task) {
- scoped_refptr<base::SingleThreadTaskRunner> task_runner(
- GetBackgroundTaskRunner());
- if (task_runner.get())
- return task_runner->PostTask(FROM_HERE, task);
- return false;
-}
-
scoped_refptr<base::SingleThreadTaskRunner>
PasswordStore::GetBackgroundTaskRunner() {
return db_thread_runner_;

Powered by Google App Engine
This is Rietveld 408576698