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

Unified Diff: chrome/browser/sync_file_system/drive_backend_v1/drive_file_sync_service.cc

Issue 190663004: [SyncFS] Drop default priority version of SyncTaskManager::ScheduleTask (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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_v1/drive_file_sync_service.cc
diff --git a/chrome/browser/sync_file_system/drive_backend_v1/drive_file_sync_service.cc b/chrome/browser/sync_file_system/drive_backend_v1/drive_file_sync_service.cc
index 0079bba957e189a51bab990f193d9870cdb7088d..a5d38daaaee9b764bb07b9d490ca90557162f356 100644
--- a/chrome/browser/sync_file_system/drive_backend_v1/drive_file_sync_service.cc
+++ b/chrome/browser/sync_file_system/drive_backend_v1/drive_file_sync_service.cc
@@ -150,7 +150,7 @@ void DriveFileSyncService::RegisterOrigin(
pending_origin_operations_.Push(origin, OriginOperation::REGISTERING);
- task_manager_->ScheduleTaskAtPriority(
+ task_manager_->ScheduleTask(
FROM_HERE,
base::Bind(&DriveFileSyncService::DoRegisterOrigin, AsWeakPtr(), origin),
SyncTaskManager::PRIORITY_HIGH,
@@ -161,7 +161,7 @@ void DriveFileSyncService::EnableOrigin(
const GURL& origin,
const SyncStatusCallback& callback) {
pending_origin_operations_.Push(origin, OriginOperation::ENABLING);
- task_manager_->ScheduleTaskAtPriority(
+ task_manager_->ScheduleTask(
FROM_HERE,
base::Bind(&DriveFileSyncService::DoEnableOrigin, AsWeakPtr(), origin),
SyncTaskManager::PRIORITY_HIGH,
@@ -172,7 +172,7 @@ void DriveFileSyncService::DisableOrigin(
const GURL& origin,
const SyncStatusCallback& callback) {
pending_origin_operations_.Push(origin, OriginOperation::DISABLING);
- task_manager_->ScheduleTaskAtPriority(
+ task_manager_->ScheduleTask(
FROM_HERE,
base::Bind(&DriveFileSyncService::DoDisableOrigin, AsWeakPtr(), origin),
SyncTaskManager::PRIORITY_HIGH,
@@ -184,7 +184,7 @@ void DriveFileSyncService::UninstallOrigin(
UninstallFlag flag,
const SyncStatusCallback& callback) {
pending_origin_operations_.Push(origin, OriginOperation::UNINSTALLING);
- task_manager_->ScheduleTaskAtPriority(
+ task_manager_->ScheduleTask(
FROM_HERE,
base::Bind(&DriveFileSyncService::DoUninstallOrigin, AsWeakPtr(),
origin, flag),
@@ -198,6 +198,7 @@ void DriveFileSyncService::ProcessRemoteChange(
FROM_HERE,
base::Bind(&DriveFileSyncService::DoProcessRemoteChange, AsWeakPtr(),
callback),
+ SyncTaskManager::PRIORITY_MED,
base::Bind(&EmptyStatusCallback));
}
@@ -303,6 +304,7 @@ void DriveFileSyncService::GetRemoteVersions(
FROM_HERE,
base::Bind(&DriveFileSyncService::DoGetRemoteVersions, AsWeakPtr(),
url, callback),
+ SyncTaskManager::PRIORITY_MED,
base::Bind(&EmptyStatusCallback));
}
@@ -314,6 +316,7 @@ void DriveFileSyncService::DownloadRemoteVersion(
FROM_HERE,
base::Bind(&DriveFileSyncService::DoDownloadRemoteVersion, AsWeakPtr(),
url, version_id, callback),
+ SyncTaskManager::PRIORITY_MED,
base::Bind(&EmptyStatusCallback));
}
@@ -333,6 +336,7 @@ void DriveFileSyncService::ApplyLocalChange(
local_file_path,
local_file_metadata,
url),
+ SyncTaskManager::PRIORITY_MED,
callback);
}

Powered by Google App Engine
This is Rietveld 408576698