cc: Generalize raster task notifications

cc/resources/gpu_raster_worker_pool.h

 // Copyright 2014 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.
 
 #ifndef CC_RESOURCES_GPU_RASTER_WORKER_POOL_H_
 #define CC_RESOURCES_GPU_RASTER_WORKER_POOL_H_
 
+#include <vector>
+
 #include "base/memory/weak_ptr.h"
 #include "cc/resources/raster_worker_pool.h"
 #include "cc/resources/rasterizer.h"
 
 namespace cc {
 class ContextProvider;
 class ResourceProvider;
 
 class CC_EXPORT GpuRasterWorkerPool : public RasterWorkerPool,
                                       public Rasterizer,
@@ skipping 18 matching lines @@
   // Overridden from RasterizerTaskClient:
   virtual RasterBuffer* AcquireBufferForRaster(RasterTask* task) OVERRIDE;
   virtual void ReleaseBufferForRaster(RasterTask* task) OVERRIDE;
 
  private:
   GpuRasterWorkerPool(base::SequencedTaskRunner* task_runner,
                       ContextProvider* context_provider,
                       ResourceProvider* resource_provider);
 
   void OnRasterFinished();
-  void OnRasterRequiredForActivationFinished();
+  void OnRasterTaskSetFinished(TaskSet task_set);
   void ScheduleRunTasksOnOriginThread();
   void RunTasksOnOriginThread();
   void RunTaskOnOriginThread(RasterizerTask* task);
 
   scoped_refptr<base::SequencedTaskRunner> task_runner_;
   scoped_ptr<TaskGraphRunner> task_graph_runner_;
   const NamespaceToken namespace_token_;
   RasterizerClient* client_;
   ContextProvider* context_provider_;
   ResourceProvider* resource_provider_;
 
   bool run_tasks_on_origin_thread_pending_;
 
   bool raster_tasks_pending_;
-  bool raster_tasks_required_for_activation_pending_;
+  RasterTaskQueue::Item::TaskSetCollection raster_task_sets_pending_;
 
   base::WeakPtrFactory<GpuRasterWorkerPool> raster_finished_weak_ptr_factory_;
 
   scoped_refptr<RasterizerTask> raster_finished_task_;
-  scoped_refptr<RasterizerTask> raster_required_for_activation_finished_task_;
+  std::vector<scoped_refptr<RasterizerTask> > task_set_finished_tasks_;

[reveman, 2014/09/05 08:38:59]

I think you can just use an array here instead.

[ernstm, 2014/09/05 21:36:10]

Done.

 
   // Task graph used when scheduling tasks and vector used to gather
   // completed tasks.
   TaskGraph graph_;
   Task::Vector completed_tasks_;
 
   base::WeakPtrFactory<GpuRasterWorkerPool> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GpuRasterWorkerPool);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_GPU_RASTER_WORKER_POOL_H_