DM: some refactoring

dm/DMChecksumTask.h

-#ifndef DMComparisonTask_DEFINED
-#define DMComparisonTask_DEFINED
+#ifndef DMChecksumTask_DEFINED
+#define DMChecksumTask_DEFINED
 
 #include "DMTask.h"
 #include "SkBitmap.h"
 #include "SkString.h"
 #include "gm_expectations.h"
 
 namespace DM {
 
-// We use ComparisonTask to move CPU-bound comparison work of GpuTasks back to
-// the main thread pool, where we probably have more threads available.
-
-class ComparisonTask : public Task {
+// ChecksumTask compares an SkBitmap against some Expectations.
+// Moving this off the GPU threadpool is a nice (~30%) runtime win.
+class ChecksumTask : public Task {

[epoger, 2013/11/26 17:55:22]

Not a big deal either way, but IMO "ComparisonTask" is still more descriptive
(based on the description above, and the arguments passed into the constructor)

[mtklein, 2013/11/26 18:25:30]

Yeah, wanted something to make it distinct from the other bitmap-to-bitmap
comparison tasks (replay, pipe, etc.)

 public:
-    ComparisonTask(const Task& parent, skiagm::Expectations, SkBitmap);
+    ChecksumTask(const Task& parent, skiagm::Expectations, SkBitmap);
 
     virtual void draw() SK_OVERRIDE;
     virtual bool usesGpu() const SK_OVERRIDE { return false; }
     virtual bool shouldSkip() const SK_OVERRIDE { return false; }
     virtual SkString name() const SK_OVERRIDE { return fName; }
 
 private:
     const SkString fName;
     const skiagm::Expectations fExpectations;
     const SkBitmap fBitmap;
 };
 
 }  // namespace DM
 
-#endif  // DMComparisonTask_DEFINED
+#endif  // DMChecksumTask_DEFINED