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

Unified Diff: chrome/browser/sync_file_system/drive_backend/sync_worker.h

Issue 288193002: [SyncFS] Construct and destruct sync_worker in worker_task_runner (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Work for a nit, from offline chat Created 6 years, 7 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/sync_file_system/drive_backend/sync_worker.h
diff --git a/chrome/browser/sync_file_system/drive_backend/sync_worker.h b/chrome/browser/sync_file_system/drive_backend/sync_worker.h
index 36742c77a6291d561df0cec7dc0d442ffad7ca4a..933c2c41cb50075d7387db51eb1a2d3e0d62f869 100644
--- a/chrome/browser/sync_file_system/drive_backend/sync_worker.h
+++ b/chrome/browser/sync_file_system/drive_backend/sync_worker.h
@@ -73,12 +73,10 @@ class SyncWorker : public SyncTaskManager::Client {
virtual ~Observer() {}
};
- static scoped_ptr<SyncWorker> CreateOnWorker(
- const base::FilePath& base_dir,
- Observer* observer,
- const base::WeakPtr<ExtensionServiceInterface>& extension_service,
- scoped_ptr<SyncEngineContext> sync_engine_context,
- leveldb::Env* env_override);
+ SyncWorker(const base::FilePath& base_dir,
+ const base::WeakPtr<ExtensionServiceInterface>& extension_service,
+ scoped_ptr<SyncEngineContext> sync_engine_context,
+ leveldb::Env* env_override);
virtual ~SyncWorker();
@@ -140,11 +138,6 @@ class SyncWorker : public SyncTaskManager::Client {
friend class DriveBackendSyncTest;
friend class SyncEngineTest;
- SyncWorker(const base::FilePath& base_dir,
- const base::WeakPtr<ExtensionServiceInterface>& extension_service,
- scoped_ptr<SyncEngineContext> sync_engine_context,
- leveldb::Env* env_override);
-
void DoDisableApp(const std::string& app_id,
const SyncStatusCallback& callback);
void DoEnableApp(const std::string& app_id,

Powered by Google App Engine
This is Rietveld 408576698