cc: Refactor RasterWorkerPool for better testing support.

This makes a number of RasterWorkerPool changes that are necessary
to implement proper performance tests and replaces existing tests
with more useful tests using realistic raster task numbers.

RasterWorkerPool is modified to allow tests to use a different
TaskGraphRunner instance. PixelBufferRasterWorkerPool
implementation is adjusted so repeated calls to ScheduleTasks
doesn't flood the message loop with canceled tasks.

BUG=246546

Review URL: https://codereview.chromium.org/150783004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249959 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 3 deletions

diff --git a/cc/resources/task_graph_runner_perftest.cc b/cc/resources/task_graph_runner_perftest.cc
index 7590450..71933f5 100644
--- a/cc/resources/task_graph_runner_perftest.cc
+++ b/cc/resources/task_graph_runner_perftest.cc
@@ -130,11 +130,11 @@ class TaskGraphRunnerPerfTest : public testing::Test {
                                      int num_top_level_tasks,
                                      int num_tasks,
                                      int num_leaf_tasks) {
-    const int kNumVersions = 2;
+    const size_t kNumVersions = 2;
     PerfTaskImpl::Vector top_level_tasks[kNumVersions];
     PerfTaskImpl::Vector tasks[kNumVersions];
     PerfTaskImpl::Vector leaf_tasks[kNumVersions];
-    for (int i = 0; i < kNumVersions; ++i) {
+    for (size_t i = 0; i < kNumVersions; ++i) {
       CreateTasks(num_top_level_tasks, &top_level_tasks[i]);
       CreateTasks(num_tasks, &tasks[i]);
       CreateTasks(num_leaf_tasks, &leaf_tasks[i]);
@@ -145,7 +145,7 @@ class TaskGraphRunnerPerfTest : public testing::Test {
     internal::TaskGraph graph;
     internal::Task::Vector completed_tasks;
 
-    int count = 0;
+    size_t count = 0;
     timer_.Reset();
     do {
       graph.Reset();