| Index: cc/resources/pixel_buffer_raster_worker_pool.cc
|
| diff --git a/cc/resources/pixel_buffer_raster_worker_pool.cc b/cc/resources/pixel_buffer_raster_worker_pool.cc
|
| index ec65214c15ee9232bb7ef71b283ec727ab98a89e..d559d7b20999d84d54747de87c138d9d79f8606c 100644
|
| --- a/cc/resources/pixel_buffer_raster_worker_pool.cc
|
| +++ b/cc/resources/pixel_buffer_raster_worker_pool.cc
|
| @@ -91,9 +91,10 @@ bool WasCanceled(const internal::RasterWorkerPoolTask* task) {
|
|
|
| PixelBufferRasterWorkerPool::PixelBufferRasterWorkerPool(
|
| ResourceProvider* resource_provider,
|
| + ContextProvider* context_provider,
|
| size_t num_threads,
|
| size_t max_transfer_buffer_usage_bytes)
|
| - : RasterWorkerPool(resource_provider, num_threads),
|
| + : RasterWorkerPool(resource_provider, context_provider, num_threads),
|
| shutdown_(false),
|
| scheduled_raster_task_count_(0),
|
| bytes_pending_upload_(0),
|
| @@ -118,7 +119,7 @@ PixelBufferRasterWorkerPool::~PixelBufferRasterWorkerPool() {
|
| void PixelBufferRasterWorkerPool::Shutdown() {
|
| shutdown_ = true;
|
| RasterWorkerPool::Shutdown();
|
| - RasterWorkerPool::CheckForCompletedTasks();
|
| + CheckForCompletedWorkerTasks();
|
| CheckForCompletedUploads();
|
| check_for_completed_raster_tasks_callback_.Cancel();
|
| check_for_completed_raster_tasks_pending_ = false;
|
| @@ -151,6 +152,7 @@ void PixelBufferRasterWorkerPool::ScheduleTasks(RasterTask::Queue* queue) {
|
|
|
| // Build new pixel buffer task set.
|
| TaskMap new_pixel_buffer_tasks;
|
| + RasterTaskVector gpu_raster_tasks;
|
| for (RasterTaskVector::const_iterator it = raster_tasks().begin();
|
| it != raster_tasks().end(); ++it) {
|
| internal::RasterWorkerPoolTask* task = it->get();
|
| @@ -158,6 +160,11 @@ void PixelBufferRasterWorkerPool::ScheduleTasks(RasterTask::Queue* queue) {
|
| DCHECK(!task->HasCompleted());
|
| DCHECK(!task->WasCanceled());
|
|
|
| + if (task->use_gpu_rasterization()) {
|
| + gpu_raster_tasks.push_back(task);
|
| + continue;
|
| + }
|
| +
|
| new_pixel_buffer_tasks[task] = pixel_buffer_tasks_[task];
|
| pixel_buffer_tasks_.erase(task);
|
|
|
| @@ -190,7 +197,7 @@ void PixelBufferRasterWorkerPool::ScheduleTasks(RasterTask::Queue* queue) {
|
| // |tasks_required_for_activation_| contains all tasks that need to
|
| // complete before we can send a "ready to activate" signal. Tasks
|
| // that have already completed should not be part of this set.
|
| - for (TaskDeque::const_iterator it = completed_tasks_.begin();
|
| + for (RasterTaskDeque::const_iterator it = completed_tasks_.begin();
|
| it != completed_tasks_.end() && !tasks_required_for_activation_.empty();
|
| ++it) {
|
| tasks_required_for_activation_.erase(*it);
|
| @@ -201,7 +208,7 @@ void PixelBufferRasterWorkerPool::ScheduleTasks(RasterTask::Queue* queue) {
|
| // Check for completed tasks when ScheduleTasks() is called as
|
| // priorities might have changed and this maximizes the number
|
| // of top priority tasks that are scheduled.
|
| - RasterWorkerPool::CheckForCompletedTasks();
|
| + CheckForCompletedWorkerTasks();
|
| CheckForCompletedUploads();
|
| FlushUploads();
|
|
|
| @@ -214,6 +221,8 @@ void PixelBufferRasterWorkerPool::ScheduleTasks(RasterTask::Queue* queue) {
|
| check_for_completed_raster_tasks_pending_ = false;
|
| ScheduleCheckForCompletedRasterTasks();
|
|
|
| + RunGpuRasterTasks(gpu_raster_tasks);
|
| +
|
| TRACE_EVENT_ASYNC_STEP_INTO1(
|
| "cc", "ScheduledTasks", this, StateName(),
|
| "state", TracedValue::FromValue(StateAsValue().release()));
|
| @@ -234,7 +243,7 @@ void PixelBufferRasterWorkerPool::CheckForCompletedTasks() {
|
| CheckForCompletedUploads();
|
| FlushUploads();
|
|
|
| - TaskDeque completed_tasks;
|
| + RasterTaskDeque completed_tasks;
|
| completed_tasks_.swap(completed_tasks);
|
|
|
| while (!completed_tasks.empty()) {
|
| @@ -288,7 +297,7 @@ void PixelBufferRasterWorkerPool::FlushUploads() {
|
| }
|
|
|
| void PixelBufferRasterWorkerPool::CheckForCompletedUploads() {
|
| - TaskDeque tasks_with_completed_uploads;
|
| + RasterTaskDeque tasks_with_completed_uploads;
|
|
|
| // First check if any have completed.
|
| while (!tasks_with_pending_upload_.empty()) {
|
| @@ -308,8 +317,8 @@ void PixelBufferRasterWorkerPool::CheckForCompletedUploads() {
|
| shutdown_ || client()->ShouldForceTasksRequiredForActivationToComplete();
|
|
|
| if (should_force_some_uploads_to_complete) {
|
| - TaskDeque tasks_with_uploads_to_force;
|
| - TaskDeque::iterator it = tasks_with_pending_upload_.begin();
|
| + RasterTaskDeque tasks_with_uploads_to_force;
|
| + RasterTaskDeque::iterator it = tasks_with_pending_upload_.begin();
|
| while (it != tasks_with_pending_upload_.end()) {
|
| internal::RasterWorkerPoolTask* task = it->get();
|
| DCHECK(pixel_buffer_tasks_.find(task) != pixel_buffer_tasks_.end());
|
| @@ -328,9 +337,9 @@ void PixelBufferRasterWorkerPool::CheckForCompletedUploads() {
|
|
|
| // Force uploads in reverse order. Since forcing can cause a wait on
|
| // all previous uploads, we would rather wait only once downstream.
|
| - for (TaskDeque::reverse_iterator it = tasks_with_uploads_to_force.rbegin();
|
| - it != tasks_with_uploads_to_force.rend();
|
| - ++it) {
|
| + for (RasterTaskDeque::reverse_iterator it =
|
| + tasks_with_uploads_to_force.rbegin();
|
| + it != tasks_with_uploads_to_force.rend(); ++it) {
|
| resource_provider()->ForceSetPixelsToComplete((*it)->resource()->id());
|
| has_performed_uploads_since_last_flush_ = true;
|
| }
|
| @@ -383,7 +392,7 @@ void PixelBufferRasterWorkerPool::CheckForCompletedRasterTasks() {
|
| check_for_completed_raster_tasks_callback_.Cancel();
|
| check_for_completed_raster_tasks_pending_ = false;
|
|
|
| - RasterWorkerPool::CheckForCompletedTasks();
|
| + CheckForCompletedWorkerTasks();
|
| CheckForCompletedUploads();
|
| FlushUploads();
|
|
|
| @@ -592,6 +601,7 @@ void PixelBufferRasterWorkerPool::OnRasterTaskCompleted(
|
| "was_canceled", was_canceled,
|
| "needs_upload", needs_upload);
|
|
|
| + DCHECK(!task->use_gpu_rasterization());
|
| DCHECK(pixel_buffer_tasks_.find(task.get()) != pixel_buffer_tasks_.end());
|
|
|
| // Balanced with MapPixelBuffer() call in ScheduleMoreTasks().
|
|
|