cc: Refactor TileTaskWorkerPool.

cc/tiles/tile_manager_perftest.cc

 // Copyright 2013 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 <stddef.h>
 #include <stdint.h>
 
 #include "base/lazy_instance.h"
 #include "base/location.h"
 #include "base/thread_task_runner_handle.h"
@@ skipping 23 matching lines @@
 namespace {
 
 static const int kTimeLimitMillis = 2000;
 static const int kWarmupRuns = 5;
 static const int kTimeCheckInterval = 10;
 
 class FakeTileTaskWorkerPoolImpl : public TileTaskWorkerPool,
                                    public RasterBufferProvider {
  public:
   // Overridden from TileTaskWorkerPool:
+  void BarrierToSyncResources() override {}
+  ResourceFormat GetResourceFormat(bool must_support_alpha) const override {
+    return RGBA_8888;
+  }
+  bool GetResourceRequiresSwizzle(bool must_support_alpha) const override {
+    return ResourceFormatRequiresSwizzle(GetResourceFormat(must_support_alpha));
+  }
+  RasterBufferProvider* AsRasterBufferProvider() override { return this; }
+
+  // Overridden from RasterBufferProvider:
+  std::unique_ptr<RasterBuffer> AcquireBufferForRaster(
+      const Resource* resource,
+      uint64_t new_content_id,
+      uint64_t previous_content_id) override {
+    return nullptr;
+  }
+  void ReleaseBufferForRaster(std::unique_ptr<RasterBuffer> buffer) override {}
+};
+
+class FakeTileTaskManager : public TileTaskManager {

[ericrk, 2016/04/25 20:56:07]

This looks like the same code from fake_tile_manager.cc... can we make a
fake_tile_task_manager.h/cc and share this?

+ public:
+  FakeTileTaskManager()
+      : TileTaskManager(base::WrapUnique<TileTaskWorkerPool>(
+            new FakeTileTaskWorkerPoolImpl)) {}
+  ~FakeTileTaskManager() override {}
   void Shutdown() override {}
   void ScheduleTasks(TaskGraph* graph) override {
     for (auto& node : graph->nodes) {
       TileTask* task = static_cast<TileTask*>(node.task);
 
       task->WillSchedule();
-      task->ScheduleOnOriginThread(this);
+      task->ScheduleOnOriginThread(
+          tile_task_worker_pool_->AsRasterBufferProvider());
       task->DidSchedule();
 
       completed_tasks_.push_back(task);
     }
   }
   void CheckForCompletedTasks() override {
     for (TileTask::Vector::iterator it = completed_tasks_.begin();
          it != completed_tasks_.end(); ++it) {
       TileTask* task = it->get();
 
       task->WillComplete();
-      task->CompleteOnOriginThread(this);
+      task->CompleteOnOriginThread(
+          tile_task_worker_pool_->AsRasterBufferProvider());
       task->DidComplete();
     }
     completed_tasks_.clear();
   }
-  ResourceFormat GetResourceFormat(bool must_support_alpha) const override {
-    return RGBA_8888;
+
+  // Get TileTaskWorkerPool.
+  TileTaskWorkerPool* GetTileTaskWorkerPool() const override {
+    return tile_task_worker_pool_.get();
   }
-  bool GetResourceRequiresSwizzle(bool must_support_alpha) const override {
-    return ResourceFormatRequiresSwizzle(GetResourceFormat(must_support_alpha));
-  }
-  RasterBufferProvider* AsRasterBufferProvider() override { return this; }
-
-  // Overridden from RasterBufferProvider:
-  std::unique_ptr<RasterBuffer> AcquireBufferForRaster(
-      const Resource* resource,
-      uint64_t new_content_id,
-      uint64_t previous_content_id) override {
-    return nullptr;
-  }
-  void ReleaseBufferForRaster(std::unique_ptr<RasterBuffer> buffer) override {}
 
  private:
   TileTask::Vector completed_tasks_;
 };
-base::LazyInstance<FakeTileTaskWorkerPoolImpl> g_fake_tile_task_worker_pool =
+
+base::LazyInstance<FakeTileTaskManager> g_fake_tile_task_manager =
     LAZY_INSTANCE_INITIALIZER;
 
 class TileManagerPerfTest : public testing::Test {
  public:
   TileManagerPerfTest()
       : memory_limit_policy_(ALLOW_ANYTHING),
         max_tiles_(10000),
         id_(7),
         task_runner_provider_(base::ThreadTaskRunnerHandle::Get()),
         output_surface_(FakeOutputSurface::Create3d()),
@@ skipping 22 matching lines @@
   }
 
   void SetUp() override {
     InitializeRenderer();
     SetTreePriority(SAME_PRIORITY_FOR_BOTH_TREES);
   }
 
   virtual void InitializeRenderer() {
     host_impl_.SetVisible(true);
     host_impl_.InitializeRenderer(output_surface_.get());
-    tile_manager()->SetTileTaskWorkerPoolForTesting(
-        g_fake_tile_task_worker_pool.Pointer());
+    tile_manager()->SetTileTaskManagerForTesting(
+        g_fake_tile_task_manager.Pointer());
   }
 
   void SetupDefaultTrees(const gfx::Size& layer_bounds) {
     scoped_refptr<FakeRasterSource> pending_raster_source =
         FakeRasterSource::CreateFilled(layer_bounds);
     scoped_refptr<FakeRasterSource> active_raster_source =
         FakeRasterSource::CreateFilled(layer_bounds);
 
     SetupTrees(pending_raster_source, active_raster_source);
   }
@@ skipping 336 matching lines @@
   RunEvictionQueueConstructAndIterateTest("10_64", 10, 64);
   RunEvictionQueueConstructAndIterateTest("10_128", 10, 128);
   RunEvictionQueueConstructAndIterateTest("50_16", 50, 16);
   RunEvictionQueueConstructAndIterateTest("50_32", 50, 32);
   RunEvictionQueueConstructAndIterateTest("50_64", 50, 64);
   RunEvictionQueueConstructAndIterateTest("50_128", 50, 128);
 }
 
 }  // namespace
 }  // namespace cc