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

Unified Diff: chrome/browser/sync_file_system/sync_task_manager.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/sync_task_manager.cc
diff --git a/chrome/browser/sync_file_system/sync_task_manager.cc b/chrome/browser/sync_file_system/sync_task_manager.cc
index 8aa330ece76c0d9e746a8dafce09b6dbc919159e..e2533067f74d07665aa5deca37d9ec174a2cf99a 100644
--- a/chrome/browser/sync_file_system/sync_task_manager.cc
+++ b/chrome/browser/sync_file_system/sync_task_manager.cc
@@ -86,34 +86,20 @@ void SyncTaskManager::Initialize(SyncStatusCode status) {
void SyncTaskManager::ScheduleTask(
const tracked_objects::Location& from_here,
const Task& task,
- const SyncStatusCallback& callback) {
- ScheduleTaskAtPriority(from_here, task, PRIORITY_MED, callback);
-}
-
-void SyncTaskManager::ScheduleSyncTask(
- const tracked_objects::Location& from_here,
- scoped_ptr<SyncTask> task,
- const SyncStatusCallback& callback) {
- ScheduleSyncTaskAtPriority(from_here, task.Pass(), PRIORITY_MED, callback);
-}
-
-void SyncTaskManager::ScheduleTaskAtPriority(
- const tracked_objects::Location& from_here,
- const Task& task,
Priority priority,
const SyncStatusCallback& callback) {
scoped_ptr<TaskToken> token(GetToken(from_here));
if (!token) {
PushPendingTask(
base::Bind(&SyncTaskManager::ScheduleTask, AsWeakPtr(), from_here,
- task, callback),
+ task, priority, callback),
priority);
return;
}
task.Run(CreateCompletionCallback(token.Pass(), callback));
}
-void SyncTaskManager::ScheduleSyncTaskAtPriority(
+void SyncTaskManager::ScheduleSyncTask(
const tracked_objects::Location& from_here,
scoped_ptr<SyncTask> task,
Priority priority,
@@ -122,7 +108,7 @@ void SyncTaskManager::ScheduleSyncTaskAtPriority(
if (!token) {
PushPendingTask(
base::Bind(&SyncTaskManager::ScheduleSyncTask, AsWeakPtr(), from_here,
- base::Passed(&task), callback),
+ base::Passed(&task), priority, callback),
priority);
return;
}
« no previous file with comments | « chrome/browser/sync_file_system/sync_task_manager.h ('k') | chrome/browser/sync_file_system/sync_task_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698