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: chrome/browser/sync_file_system/drive_backend/sync_engine.cc

Issue 306813002: [SyncFS] Make routines using metadata database async (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase 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.cc
diff --git a/chrome/browser/sync_file_system/drive_backend/sync_engine.cc b/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
index b8d9c17d8d39d903dafcc51b577edd036da883b1..ceb73aca69c9d1a37446d6b4535e785c56705ed2 100644
--- a/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
+++ b/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
@@ -394,14 +394,10 @@ void SyncEngine::SetSyncEnabled(bool enabled) {
}
void SyncEngine::PromoteDemotedChanges() {
- MetadataDatabase* metadata_db = GetMetadataDatabase();
- if (metadata_db && metadata_db->HasLowPriorityDirtyTracker()) {
- metadata_db->PromoteLowerPriorityTrackersToNormal();
- FOR_EACH_OBSERVER(
- Observer,
- service_observers_,
- OnRemoteChangeQueueUpdated(metadata_db->CountDirtyTracker()));
- }
+ worker_task_runner_->PostTask(
+ FROM_HERE,
+ base::Bind(&SyncWorker::PromoteDemotedChanges,
+ base::Unretained(sync_worker_.get())));
}
void SyncEngine::ApplyLocalChange(
@@ -422,10 +418,6 @@ void SyncEngine::ApplyLocalChange(
FROM_HERE, callback)));
}
-SyncTaskManager* SyncEngine::GetSyncTaskManagerForTesting() {
- return sync_worker_->GetSyncTaskManager();
-}
-
void SyncEngine::OnNotificationReceived() {
worker_task_runner_->PostTask(
FROM_HERE,
@@ -470,11 +462,6 @@ drive::DriveUploaderInterface* SyncEngine::GetDriveUploader() {
return drive_uploader_.get();
}
-MetadataDatabase* SyncEngine::GetMetadataDatabase() {
- // TODO(peria): Post task
- return sync_worker_->GetMetadataDatabase();
-}
-
SyncEngine::SyncEngine(
scoped_ptr<drive::DriveServiceInterface> drive_service,
scoped_ptr<drive::DriveUploaderInterface> drive_uploader,
@@ -519,11 +506,11 @@ void SyncEngine::UpdateServiceState(RemoteServiceState state,
OnRemoteServiceStateUpdated(state, description));
}
-void SyncEngine::UpdateRegisteredApps() {
+void SyncEngine::UpdateRegisteredAppsForTesting() {
if (!extension_service_)
return;
- MetadataDatabase* metadata_db = GetMetadataDatabase();
+ MetadataDatabase* metadata_db = sync_worker_->GetMetadataDatabase();
DCHECK(metadata_db);
std::vector<std::string> app_ids;
metadata_db->GetRegisteredAppIDs(&app_ids);

Powered by Google App Engine
This is Rietveld 408576698