cc: Remove ScheduleOnOriginThread() and CompleteOnOriginThread().

cc/tiles/gpu_image_decode_controller_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/tiles/gpu_image_decode_controller.h"
 
 #include "cc/playback/draw_image.h"
 #include "cc/raster/tile_task_runner.h"
 #include "cc/test/test_context_provider.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 13 matching lines @@
   matrix.setScale(scale.width(), scale.height());
 
   if (!is_decomposable) {
     // Perspective is not decomposable, add it.
     matrix[SkMatrix::kMPersp0] = 0.1f;
   }
 
   return matrix;
 }
 
-void ScheduleTask(ImageDecodeTask* task) {
-  task->WillSchedule();
-  task->ScheduleOnOriginThread(nullptr);
-  task->DidSchedule();
-}
-
 void RunTask(ImageDecodeTask* task) {
   task->WillRun();
   task->RunOnWorkerThread();
   task->DidRun();
 }
 
-void CompleteTask(ImageDecodeTask* task) {
-  task->WillComplete();
-  task->CompleteOnOriginThread(nullptr);
+void CompleteTask(GpuImageDecodeController* controller, ImageDecodeTask* task) {
+  if (task->GetTaskTypeId() == TASK_TYPE_IMAGE_DECODE)
+    controller->ImageDecodeTaskCompleted(task);
+  else if (task->GetTaskTypeId() == TASK_TYPE_IMAGE_UPLOAD)
+    controller->ImageUploadTaskCompleted(task);
+
   task->DidComplete();
 }
 
-void ProcessTask(ImageDecodeTask* task) {
-  ScheduleTask(task);
+void ProcessTask(GpuImageDecodeController* controller, ImageDecodeTask* task) {
   RunTask(task);
-  CompleteTask(task);
+  CompleteTask(controller, task);
 }
 
 TEST(GpuImageDecodeControllerTest, GetTaskForImageSameImage) {
   auto context_provider = TestContextProvider::Create();
   context_provider->BindToCurrentThread();
   GpuImageDecodeController controller(context_provider.get(),
                                       ResourceFormat::RGBA_8888);
   skia::RefPtr<SkImage> image = CreateImage(100, 100);
   bool is_decomposable = true;
   SkFilterQuality quality = kHigh_SkFilterQuality;
@@ skipping 10 matching lines @@
 
   DrawImage another_draw_image(
       image.get(), SkIRect::MakeWH(image->width(), image->height()), quality,
       CreateMatrix(SkSize::Make(1.5f, 1.5f), is_decomposable));
   scoped_refptr<ImageDecodeTask> another_task;
   need_unref = controller.GetTaskForImageAndRef(
       another_draw_image, prepare_tiles_id, &another_task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(task.get() == another_task.get());
 
-  ProcessTask(task->dependency().get());
-  ProcessTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
+  ProcessTask(&controller, task.get());
 
   controller.UnrefImage(draw_image);
   controller.UnrefImage(draw_image);
 }
 
 TEST(GpuImageDecodeControllerTest, GetTaskForImageDifferentImage) {
   auto context_provider = TestContextProvider::Create();
   context_provider->BindToCurrentThread();
   GpuImageDecodeController controller(context_provider.get(),
                                       ResourceFormat::RGBA_8888);
@@ skipping 17 matching lines @@
       second_image.get(),
       SkIRect::MakeWH(second_image->width(), second_image->height()), quality,
       CreateMatrix(SkSize::Make(0.25f, 0.25f), is_decomposable));
   scoped_refptr<ImageDecodeTask> second_task;
   need_unref = controller.GetTaskForImageAndRef(second_draw_image,
                                                 prepare_tiles_id, &second_task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(second_task);
   EXPECT_TRUE(first_task.get() != second_task.get());
 
-  ProcessTask(first_task->dependency().get());
-  ProcessTask(first_task.get());
-  ProcessTask(second_task->dependency().get());
-  ProcessTask(second_task.get());
+  ProcessTask(&controller, first_task->dependency().get());
+  ProcessTask(&controller, first_task.get());
+  ProcessTask(&controller, second_task->dependency().get());
+  ProcessTask(&controller, second_task.get());
 
   controller.UnrefImage(first_draw_image);
   controller.UnrefImage(second_draw_image);
 }
 
 TEST(GpuImageDecodeControllerTest, GetTaskForImageAlreadyDecoded) {
   auto context_provider = TestContextProvider::Create();
   context_provider->BindToCurrentThread();
   GpuImageDecodeController controller(context_provider.get(),
                                       ResourceFormat::RGBA_8888);
   bool is_decomposable = true;
   uint64_t prepare_tiles_id = 1;
   SkFilterQuality quality = kHigh_SkFilterQuality;
 
   skia::RefPtr<SkImage> image = CreateImage(100, 100);
   DrawImage draw_image(
       image.get(), SkIRect::MakeWH(image->width(), image->height()), quality,
       CreateMatrix(SkSize::Make(0.5f, 0.5f), is_decomposable));
   scoped_refptr<ImageDecodeTask> task;
   bool need_unref =
       controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(task);
   EXPECT_TRUE(task->dependency());
 
-  ProcessTask(task->dependency().get());
-  ScheduleTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
   RunTask(task.get());
 
   scoped_refptr<ImageDecodeTask> another_task;
   need_unref = controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id,
                                                 &another_task);
   EXPECT_TRUE(need_unref);
   EXPECT_FALSE(another_task);
 
-  CompleteTask(task.get());
+  CompleteTask(&controller, task.get());
 
   controller.UnrefImage(draw_image);
   controller.UnrefImage(draw_image);
 }
 
 TEST(GpuImageDecodeControllerTest, GetTaskForImageCanceledGetsNewTask) {
   auto context_provider = TestContextProvider::Create();
   context_provider->BindToCurrentThread();
   GpuImageDecodeController controller(context_provider.get(),
                                       ResourceFormat::RGBA_8888);
   bool is_decomposable = true;
   uint64_t prepare_tiles_id = 1;
   SkFilterQuality quality = kHigh_SkFilterQuality;
 
   skia::RefPtr<SkImage> image = CreateImage(100, 100);
   DrawImage draw_image(
       image.get(), SkIRect::MakeWH(image->width(), image->height()), quality,
       CreateMatrix(SkSize::Make(0.5f, 0.5f), is_decomposable));
   scoped_refptr<ImageDecodeTask> task;
   bool need_unref =
       controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(task);
 
-  ProcessTask(task->dependency().get());
-  ScheduleTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
 
   scoped_refptr<ImageDecodeTask> another_task;
   need_unref = controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id,
                                                 &another_task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(another_task.get() == task.get());
 
   // Didn't run the task, complete it (it was canceled).
-  CompleteTask(task.get());
+  CompleteTask(&controller, task.get());
 
   // Fully cancel everything (so the raster would unref things).
   controller.UnrefImage(draw_image);
   controller.UnrefImage(draw_image);
 
   // Here a new task is created.
   scoped_refptr<ImageDecodeTask> third_task;
   need_unref = controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id,
                                                 &third_task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(third_task);
   EXPECT_FALSE(third_task.get() == task.get());
 
-  ProcessTask(third_task->dependency().get());
-  ProcessTask(third_task.get());
+  ProcessTask(&controller, third_task->dependency().get());
+  ProcessTask(&controller, third_task.get());
 
   controller.UnrefImage(draw_image);
 }
 
 TEST(GpuImageDecodeControllerTest,
      GetTaskForImageCanceledWhileReffedGetsNewTask) {
   auto context_provider = TestContextProvider::Create();
   context_provider->BindToCurrentThread();
   GpuImageDecodeController controller(context_provider.get(),
                                       ResourceFormat::RGBA_8888);
   bool is_decomposable = true;
   uint64_t prepare_tiles_id = 1;
   SkFilterQuality quality = kHigh_SkFilterQuality;
 
   skia::RefPtr<SkImage> image = CreateImage(100, 100);
   DrawImage draw_image(
       image.get(), SkIRect::MakeWH(image->width(), image->height()), quality,
       CreateMatrix(SkSize::Make(0.5f, 0.5f), is_decomposable));
   scoped_refptr<ImageDecodeTask> task;
   bool need_unref =
       controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(task);
 
-  ProcessTask(task->dependency().get());
-  ScheduleTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
 
   scoped_refptr<ImageDecodeTask> another_task;
   need_unref = controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id,
                                                 &another_task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(another_task.get() == task.get());
 
   // Didn't run the task, complete it (it was canceled).
-  CompleteTask(task.get());
+  CompleteTask(&controller, task.get());
 
   // Note that here, everything is reffed, but a new task is created. This is
   // possible with repeated schedule/cancel operations.
   scoped_refptr<ImageDecodeTask> third_task;
   need_unref = controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id,
                                                 &third_task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(third_task);
   EXPECT_FALSE(third_task.get() == task.get());
 
-  ProcessTask(third_task->dependency().get());
-  ProcessTask(third_task.get());
+  ProcessTask(&controller, third_task->dependency().get());
+  ProcessTask(&controller, third_task.get());
 
   // 3 Unrefs!
   controller.UnrefImage(draw_image);
   controller.UnrefImage(draw_image);
   controller.UnrefImage(draw_image);
 }
 
 TEST(GpuImageDecodeControllerTest, GetDecodedImageForDraw) {
   auto context_provider = TestContextProvider::Create();
   context_provider->BindToCurrentThread();
   GpuImageDecodeController controller(context_provider.get(),
                                       ResourceFormat::RGBA_8888);
   bool is_decomposable = true;
   uint64_t prepare_tiles_id = 1;
   SkFilterQuality quality = kHigh_SkFilterQuality;
 
   skia::RefPtr<SkImage> image = CreateImage(100, 100);
   DrawImage draw_image(
       image.get(), SkIRect::MakeWH(image->width(), image->height()), quality,
       CreateMatrix(SkSize::Make(0.5f, 0.5f), is_decomposable));
   scoped_refptr<ImageDecodeTask> task;
   bool need_unref =
       controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(task);
 
-  ProcessTask(task->dependency().get());
-  ProcessTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
+  ProcessTask(&controller, task.get());
 
   // Must hold context lock before calling GetDecodedImageForDraw /
   // DrawWithImageFinished.
   ContextProvider::ScopedContextLock context_lock(context_provider.get());
   DecodedDrawImage decoded_draw_image =
       controller.GetDecodedImageForDraw(draw_image);
   EXPECT_TRUE(decoded_draw_image.image());
   EXPECT_TRUE(decoded_draw_image.image()->isTextureBacked());
   EXPECT_FALSE(decoded_draw_image.is_at_raster_decode());
 
@@ skipping 13 matching lines @@
   skia::RefPtr<SkImage> image = CreateImage(1, 24000);
   DrawImage draw_image(
       image.get(), SkIRect::MakeWH(image->width(), image->height()), quality,
       CreateMatrix(SkSize::Make(0.5f, 0.5f), is_decomposable));
   scoped_refptr<ImageDecodeTask> task;
   bool need_unref =
       controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
   EXPECT_TRUE(need_unref);
   EXPECT_TRUE(task);
 
-  ProcessTask(task->dependency().get());
-  ProcessTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
+  ProcessTask(&controller, task.get());
 
   // Must hold context lock before calling GetDecodedImageForDraw /
   // DrawWithImageFinished.
   ContextProvider::ScopedContextLock context_lock(context_provider.get());
   DecodedDrawImage decoded_draw_image =
       controller.GetDecodedImageForDraw(draw_image);
   EXPECT_TRUE(decoded_draw_image.image());
   EXPECT_FALSE(decoded_draw_image.image()->isTextureBacked());
   EXPECT_FALSE(decoded_draw_image.is_at_raster_decode());
 
@@ skipping 192 matching lines @@
       image.get(), SkIRect::MakeXYWH(0, 0, image->width(), image->height()),
       quality, CreateMatrix(SkSize::Make(1.f, 1.f), is_decomposable));
 
   scoped_refptr<ImageDecodeTask> task;
   bool need_unref =
       controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
   EXPECT_NE(0u, controller.GetBytesUsedForTesting());
   EXPECT_TRUE(task);
   EXPECT_TRUE(need_unref);
 
-  ScheduleTask(task->dependency().get());
-  CompleteTask(task->dependency().get());
-  ScheduleTask(task.get());
-  CompleteTask(task.get());
+  CompleteTask(&controller, task->dependency().get());
+  CompleteTask(&controller, task.get());
 
   controller.UnrefImage(draw_image);
   EXPECT_EQ(0u, controller.GetBytesUsedForTesting());
 }
 
 TEST(GpuImageDecodeControllerTest, ShouldAggressivelyFreeResources) {
   auto context_provider = TestContextProvider::Create();
   context_provider->BindToCurrentThread();
   GpuImageDecodeController controller(context_provider.get(),
                                       ResourceFormat::RGBA_8888);
   bool is_decomposable = true;
   uint64_t prepare_tiles_id = 1;
   SkFilterQuality quality = kHigh_SkFilterQuality;
 
   skia::RefPtr<SkImage> image = CreateImage(100, 100);
   DrawImage draw_image(
       image.get(), SkIRect::MakeWH(image->width(), image->height()), quality,
       CreateMatrix(SkSize::Make(0.5f, 0.5f), is_decomposable));
   scoped_refptr<ImageDecodeTask> task;
   {
     bool need_unref =
         controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
     EXPECT_TRUE(need_unref);
     EXPECT_TRUE(task);
   }
 
-  ProcessTask(task->dependency().get());
-  ProcessTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
+  ProcessTask(&controller, task.get());
 
   controller.UnrefImage(draw_image);
 
   // We should now have data image in our cache.
   DCHECK_GT(controller.GetBytesUsedForTesting(), 0u);
 
   // Tell our controller to aggressively free resources.
   controller.SetShouldAggressivelyFreeResources(true);
   DCHECK_EQ(0u, controller.GetBytesUsedForTesting());
 
   // Attempting to upload a new image should result in at-raster decode.
   {
     bool need_unref =
         controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
     EXPECT_FALSE(need_unref);
     EXPECT_FALSE(task);
   }
 
   // We now tell the controller to not aggressively free resources. Uploads
   // should work again.
   controller.SetShouldAggressivelyFreeResources(false);
   {
     bool need_unref =
         controller.GetTaskForImageAndRef(draw_image, prepare_tiles_id, &task);
     EXPECT_TRUE(need_unref);
     EXPECT_TRUE(task);
   }
 
-  ProcessTask(task->dependency().get());
-  ProcessTask(task.get());
+  ProcessTask(&controller, task->dependency().get());
+  ProcessTask(&controller, task.get());
 
   // The image should be in our cache after un-ref.
   controller.UnrefImage(draw_image);
   DCHECK_GT(controller.GetBytesUsedForTesting(), 0u);
 }
 
 }  // namespace
 }  // namespace cc