cc: Added modifier string to clarify perf tests.

cc/resources/raster_worker_pool_perftest.cc

Index: cc/resources/raster_worker_pool_perftest.cc
diff --git a/cc/resources/raster_worker_pool_perftest.cc b/cc/resources/raster_worker_pool_perftest.cc
index 703a95da911ee4be93a66461d4e5943306761386..edbac219d7811f07373b0a1ba23eff309f74ecbc 100644
--- a/cc/resources/raster_worker_pool_perftest.cc
+++ b/cc/resources/raster_worker_pool_perftest.cc
@@ -256,7 +256,7 @@ class RasterWorkerPoolPerfTest
     raster_worker_pool_->CheckForCompletedTasks();
 
     perf_test::PrintResult("schedule_tasks",
-                           "",
+                           TestModifierString(),
                            test_name,
                            timer_.LapsPerSecond(),
                            "runs/s",
@@ -294,7 +294,7 @@ class RasterWorkerPoolPerfTest
     raster_worker_pool_->CheckForCompletedTasks();
 
     perf_test::PrintResult("schedule_alternate_tasks",
-                           "",
+                           TestModifierString(),
                            test_name,
                            timer_.LapsPerSecond(),
                            "runs/s",
@@ -328,7 +328,7 @@ class RasterWorkerPoolPerfTest
     raster_worker_pool_->CheckForCompletedTasks();
 
     perf_test::PrintResult("schedule_and_execute_tasks",
-                           "",
+                           TestModifierString(),
                            test_name,
                            timer_.LapsPerSecond(),
                            "runs/s",
@@ -336,6 +336,18 @@ class RasterWorkerPoolPerfTest
   }
 
  private:
+  std::string TestModifierString() const {
+    switch (GetParam()) {
+      case RASTER_WORKER_POOL_TYPE_PIXEL_BUFFER:
+        return std::string("_pixel_raster_worker_pool");
+      case RASTER_WORKER_POOL_TYPE_IMAGE:
+        return std::string("_image_raster_worker_pool");
+      default:

[reveman, 2014/02/14 23:40:16]

Please remove this "default:" case and move NOTREACHED() below so the compiler
will complain if we add a new type but fail to update this function.

[alokp, 2014/02/15 00:26:11]

Done.

+        NOTREACHED();
+    }
+    return std::string();
+  }
+
   scoped_refptr<TestContextProvider> context_provider_;
   FakeOutputSurfaceClient output_surface_client_;
   scoped_ptr<FakeOutputSurface> output_surface_;