Chromium Code Reviews| Index: cc/tiles/tile_task_manager.h |
| diff --git a/cc/tiles/tile_task_manager.h b/cc/tiles/tile_task_manager.h |
| index e670b415602ad33e4e75111e982e5d843fcd7447..509daf63a1337a234892c950ccbc041cff560090 100644 |
| --- a/cc/tiles/tile_task_manager.h |
| +++ b/cc/tiles/tile_task_manager.h |
| @@ -28,11 +28,10 @@ class CC_EXPORT TileTaskManager { |
| // ScheduleTasks(). |
| virtual void ScheduleTasks(TaskGraph* graph) = 0; |
| - // Check for completed tasks and dispatch reply callbacks. |
| + // Check for completed tasks. |
|
vmpstr
2016/05/21 00:02:14
you should keep "and call OnTaskCompleted on them"
prashant.n
2016/05/21 00:51:45
Ok. I'll modify.
|
| virtual void CheckForCompletedTasks() = 0; |
| - // Shutdown after canceling all previously scheduled tasks. Reply callbacks |
| - // are still guaranteed to run when CheckForCompletedTasks() is called. |
| + // Shutdown after canceling all previously scheduled tasks. |
| virtual void Shutdown() = 0; |
| // Get RasterBufferProvider. |
| @@ -61,7 +60,6 @@ class CC_EXPORT TileTaskManagerImpl : public TileTaskManager { |
| std::unique_ptr<RasterBufferProvider> raster_buffer_provider_; |
| TaskGraphRunner* task_graph_runner_; |
| const NamespaceToken namespace_token_; |
| - Task::Vector completed_tasks_; |
| private: |
| DISALLOW_COPY_AND_ASSIGN(TileTaskManagerImpl); |