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

Unified Diff: components/sync/driver/glue/browser_thread_model_worker.cc

Issue 2503423004: [Sync] Move sync's ModelSafeWorker implementations to engine/. (Closed)
Patch Set: Rebase. Created 4 years, 1 month 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/sync/driver/glue/browser_thread_model_worker.cc
diff --git a/components/sync/driver/glue/browser_thread_model_worker.cc b/components/sync/driver/glue/browser_thread_model_worker.cc
deleted file mode 100644
index 736048cf262f5f10022091f0cf6341860ac1b71e..0000000000000000000000000000000000000000
--- a/components/sync/driver/glue/browser_thread_model_worker.cc
+++ /dev/null
@@ -1,69 +0,0 @@
-// Copyright 2012 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 "components/sync/driver/glue/browser_thread_model_worker.h"
-
-#include "base/bind.h"
-#include "base/callback.h"
-#include "base/single_thread_task_runner.h"
-#include "base/synchronization/waitable_event.h"
-
-using base::SingleThreadTaskRunner;
-
-namespace syncer {
-
-BrowserThreadModelWorker::BrowserThreadModelWorker(
- const scoped_refptr<SingleThreadTaskRunner>& runner,
- ModelSafeGroup group)
- : runner_(runner), group_(group) {}
-
-SyncerError BrowserThreadModelWorker::DoWorkAndWaitUntilDoneImpl(
- const WorkCallback& work) {
- SyncerError error = UNSET;
- if (runner_->BelongsToCurrentThread()) {
- DLOG(WARNING) << "Already on thread " << runner_;
- return work.Run();
- }
-
- // Signaled when the task is deleted, i.e. after it runs or when it is
- // abandoned.
- base::WaitableEvent work_done_or_abandoned(
- base::WaitableEvent::ResetPolicy::AUTOMATIC,
- base::WaitableEvent::InitialState::NOT_SIGNALED);
-
- if (!runner_->PostTask(
- FROM_HERE,
- base::Bind(
- &BrowserThreadModelWorker::CallDoWorkAndSignalTask, this, work,
- base::Passed(syncer::ScopedEventSignal(&work_done_or_abandoned)),
- &error))) {
- DLOG(WARNING) << "Failed to post task to runner " << runner_;
- error = CANNOT_DO_WORK;
- return error;
- }
- work_done_or_abandoned.Wait();
- return error;
-}
-
-ModelSafeGroup BrowserThreadModelWorker::GetModelSafeGroup() {
- return group_;
-}
-
-bool BrowserThreadModelWorker::IsOnModelThread() {
- return runner_->BelongsToCurrentThread();
-}
-
-BrowserThreadModelWorker::~BrowserThreadModelWorker() {}
-
-void BrowserThreadModelWorker::CallDoWorkAndSignalTask(
- const WorkCallback& work,
- syncer::ScopedEventSignal scoped_event_signal,
- SyncerError* error) {
- DCHECK(runner_->BelongsToCurrentThread());
- if (!IsStopped())
- *error = work.Run();
- // The event in |scoped_event_signal| is signaled at the end of this scope.
-}
-
-} // namespace syncer

Powered by Google App Engine
This is Rietveld 408576698