Revert of SkThreadPool ~~> SkTaskGroup

dm/DMTask.cpp

 #include "DMTask.h"
 #include "DMTaskRunner.h"
 #include "SkCommonFlags.h"
 
 namespace DM {
 
 Task::Task(Reporter* reporter, TaskRunner* taskRunner)
     : fReporter(reporter)
     , fTaskRunner(taskRunner)
     , fDepth(0) {
@@ skipping 20 matching lines @@
 }
 
 void Task::start() {
     fStart = SkTime::GetMSecs();
 }
 
 void Task::finish() {
     fReporter->printStatus(this->name(), SkTime::GetMSecs() - fStart);
 }
 
-void Task::reallySpawnChild(CpuTask* task) {
-    fTaskRunner->add(task);
+void Task::spawnChildNext(CpuTask* task) {
+    fTaskRunner->addNext(task);
 }
 
 CpuTask::CpuTask(Reporter* reporter, TaskRunner* taskRunner) : Task(reporter, taskRunner) {}
 CpuTask::CpuTask(const Task& parent) : Task(parent) {}
 
 void CpuTask::run() {
     if (FLAGS_cpu && !this->shouldSkip()) {
         this->start();
         if (!FLAGS_dryRun) this->draw();
         this->finish();
     }
     SkDELETE(this);
 }
 
 void CpuTask::spawnChild(CpuTask* task) {
     // Run children serially on this (CPU) thread.  This tends to save RAM and is usually no slower.
-    // Calling reallySpawnChild() is nearly equivalent, but it'd pointlessly contend on the
-    // threadpool; reallySpawnChild() is most useful when you want to change threadpools.
+    // Calling spawnChildNext() is nearly equivalent, but it'd pointlessly contend on the
+    // threadpool; spawnChildNext() is most useful when you want to change threadpools.
     task->run();
 }
 
 GpuTask::GpuTask(Reporter* reporter, TaskRunner* taskRunner) : Task(reporter, taskRunner) {}
 
-void GpuTask::run(GrContextFactory* factory) {
+void GpuTask::run(GrContextFactory& factory) {
     if (FLAGS_gpu && !this->shouldSkip()) {
         this->start();
-        if (!FLAGS_dryRun) this->draw(factory);
+        if (!FLAGS_dryRun) this->draw(&factory);
         this->finish();
         if (FLAGS_abandonGpuContext) {
-            factory->abandonContexts();
+            factory.abandonContexts();
         }
         if (FLAGS_resetGpuContext || FLAGS_abandonGpuContext) {
-            factory->destroyContexts();
+            factory.destroyContexts();
         }
     }
     SkDELETE(this);
 }
 
 void GpuTask::spawnChild(CpuTask* task) {
-    // Spawn a new task so it runs on the CPU threadpool instead of the GPU one we're on now.
+    // Really spawn a new task so it runs on the CPU threadpool instead of the GPU one we're on now.
     // It goes on the front of the queue to minimize the time we must hold reference bitmaps in RAM.
-    this->reallySpawnChild(task);
+    this->spawnChildNext(task);
 }
 
 }  // namespace DM