Index: cc/tiles/software_image_decode_controller_unittest.cc |
diff --git a/cc/tiles/software_image_decode_controller_unittest.cc b/cc/tiles/software_image_decode_controller_unittest.cc |
index 38246b38398a20307d639ee76bde98d8a204badd..6b1414cc449388a6aaea522e73e0602162db2e21 100644 |
--- a/cc/tiles/software_image_decode_controller_unittest.cc |
+++ b/cc/tiles/software_image_decode_controller_unittest.cc |
@@ -504,9 +504,6 @@ TEST(SoftwareImageDecodeControllerTest, GetTaskForImageAlreadyDecoded) { |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
task->RunOnWorkerThread(); |
scoped_refptr<ImageDecodeTask> another_task; |
@@ -515,8 +512,7 @@ TEST(SoftwareImageDecodeControllerTest, GetTaskForImageAlreadyDecoded) { |
EXPECT_TRUE(need_unref); |
EXPECT_FALSE(another_task); |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
controller.UnrefImage(draw_image); |
@@ -539,9 +535,6 @@ TEST(SoftwareImageDecodeControllerTest, GetTaskForImageAlreadyPrerolled) { |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
task->RunOnWorkerThread(); |
scoped_refptr<ImageDecodeTask> another_task; |
@@ -550,8 +543,7 @@ TEST(SoftwareImageDecodeControllerTest, GetTaskForImageAlreadyPrerolled) { |
EXPECT_TRUE(need_unref); |
EXPECT_FALSE(another_task); |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
scoped_refptr<ImageDecodeTask> third_task; |
@@ -580,11 +572,6 @@ TEST(SoftwareImageDecodeControllerTest, GetTaskForImageCanceledGetsNewTask) { |
controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task); |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
- |
scoped_refptr<ImageDecodeTask> another_task; |
need_unref = controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, |
&another_task); |
@@ -592,8 +579,7 @@ TEST(SoftwareImageDecodeControllerTest, GetTaskForImageCanceledGetsNewTask) { |
EXPECT_TRUE(another_task.get() == task.get()); |
// Didn't run the task, complete it (it was canceled). |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
// Fully cancel everything (so the raster would unref things). |
@@ -628,10 +614,6 @@ TEST(SoftwareImageDecodeControllerTest, |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
- |
scoped_refptr<ImageDecodeTask> another_task; |
need_unref = controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, |
&another_task); |
@@ -639,8 +621,7 @@ TEST(SoftwareImageDecodeControllerTest, |
EXPECT_TRUE(another_task.get() == task.get()); |
// Didn't run the task, complete it (it was canceled). |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
// Note that here, everything is reffed, but a new task is created. This is |
@@ -674,14 +655,9 @@ TEST(SoftwareImageDecodeControllerTest, GetDecodedImageForDraw) { |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
- |
task->RunOnWorkerThread(); |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
DecodedDrawImage decoded_draw_image = |
@@ -716,14 +692,9 @@ TEST(SoftwareImageDecodeControllerTest, |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
- |
task->RunOnWorkerThread(); |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
DecodedDrawImage decoded_draw_image = |
@@ -825,14 +796,9 @@ TEST(SoftwareImageDecodeControllerTest, |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
- |
task->RunOnWorkerThread(); |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
DecodedDrawImage another_decoded_draw_image = |
@@ -879,18 +845,13 @@ TEST(SoftwareImageDecodeControllerTest, |
EXPECT_TRUE(need_unref); |
EXPECT_TRUE(task); |
- task->WillSchedule(); |
- task->ScheduleOnOriginThread(nullptr); |
- task->DidSchedule(); |
- |
// If we finish the draw here, then we will use it for the locked decode |
// instead of decoding again. |
controller.DrawWithImageFinished(draw_image, decoded_draw_image); |
task->RunOnWorkerThread(); |
- task->WillComplete(); |
- task->CompleteOnOriginThread(nullptr); |
+ controller.CompleteTask(task.get()); |
task->DidComplete(); |
DecodedDrawImage another_decoded_draw_image = |