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

Unified Diff: chrome/browser/sync_file_system/drive_backend/sync_engine_initializer.cc

Issue 293003002: [SyncFS] Remove using MessageLoop in worker thread (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: RemoteToLocalSyncer 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_engine_initializer.cc
diff --git a/chrome/browser/sync_file_system/drive_backend/sync_engine_initializer.cc b/chrome/browser/sync_file_system/drive_backend/sync_engine_initializer.cc
index 5bd7f84b0a182052e80d7242982ead2a8fe7285d..03c5008a3ba57d51c7007b3bf3bfd151751a1b19 100644
--- a/chrome/browser/sync_file_system/drive_backend/sync_engine_initializer.cc
+++ b/chrome/browser/sync_file_system/drive_backend/sync_engine_initializer.cc
@@ -77,18 +77,15 @@ ScopedVector<google_apis::FileResource> ConvertResourceEntriesToFileResources(
SyncEngineInitializer::SyncEngineInitializer(
SyncEngineContext* sync_context,
- base::SequencedTaskRunner* task_runner,
const base::FilePath& database_path,
leveldb::Env* env_override)
: sync_context_(sync_context),
env_override_(env_override),
- task_runner_(task_runner),
database_path_(database_path),
find_sync_root_retry_count_(0),
largest_change_id_(0),
weak_ptr_factory_(this) {
DCHECK(sync_context);
- DCHECK(task_runner);
}
SyncEngineInitializer::~SyncEngineInitializer() {
@@ -109,13 +106,11 @@ void SyncEngineInitializer::RunPreflight(scoped_ptr<SyncTaskToken> token) {
return;
}
- // TODO(tzik): Stop using MessageLoopProxy before moving out from UI thread.
- scoped_refptr<base::SequencedTaskRunner> worker_task_runner(
- base::MessageLoopProxy::current());
-
MetadataDatabase::Create(
- worker_task_runner.get(),
- task_runner_.get(), database_path_, env_override_,
+ sync_context_->GetWorkerTaskRunner(),
+ sync_context_->GetFileTaskRunner(),
+ database_path_,
+ env_override_,
base::Bind(&SyncEngineInitializer::DidCreateMetadataDatabase,
weak_ptr_factory_.GetWeakPtr(), base::Passed(&token)));
}

Powered by Google App Engine
This is Rietveld 408576698