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

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

Issue 2466313003: Remove ModelSafeWorker::work_done_or_stopped(). (Closed)
Patch Set: self-review 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.h
diff --git a/components/sync/driver/glue/browser_thread_model_worker.h b/components/sync/driver/glue/browser_thread_model_worker.h
index 77b22ce648fb8cae777f7c1d188aad0983cf319a..668356c5b9114ec35ef2d4ccd701d5cae72d0298 100644
--- a/components/sync/driver/glue/browser_thread_model_worker.h
+++ b/components/sync/driver/glue/browser_thread_model_worker.h
@@ -10,6 +10,7 @@
#include "base/macros.h"
#include "components/sync/base/syncer_error.h"
#include "components/sync/engine/model_safe_worker.h"
+#include "components/sync/engine/signal_event_on_delete.h"
namespace base {
class SingleThreadTaskRunner;
@@ -38,12 +39,10 @@ class BrowserThreadModelWorker : public ModelSafeWorker {
SyncerError DoWorkAndWaitUntilDoneImpl(const WorkCallback& work) override;
- // Marked pure virtual so subclasses have to override, but there is
- // an implementation that subclasses should use. This is so that
- // (subclass)::CallDoWorkAndSignalTask shows up in callstacks.
- virtual void CallDoWorkAndSignalTask(const WorkCallback& work,
- base::WaitableEvent* done,
- SyncerError* error);
+ void CallDoWorkAndSignalTask(
+ const WorkCallback& work,
+ syncer::SignalEventOnDelete signal_event_on_delete,
+ SyncerError* error);
private:
scoped_refptr<base::SingleThreadTaskRunner> runner_;

Powered by Google App Engine
This is Rietveld 408576698