Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3752)

Unified Diff: cc/resources/image_raster_worker_pool.cc

Issue 83183005: Add synthetic delay points for latency testing (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address David's comments. Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | cc/resources/pixel_buffer_raster_worker_pool.cc » ('j') | cc/resources/raster_worker_pool.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/image_raster_worker_pool.cc
diff --git a/cc/resources/image_raster_worker_pool.cc b/cc/resources/image_raster_worker_pool.cc
index 90572ab417dedc02c00d3d082ce2ca94643e3449..aebfdd2f3142c7dda6a33a30e652caee3a180d01 100644
--- a/cc/resources/image_raster_worker_pool.cc
+++ b/cc/resources/image_raster_worker_pool.cc
@@ -84,9 +84,20 @@ void ImageRasterWorkerPool::ScheduleTasks(RasterTask::Queue* queue) {
unsigned priority = 0u;
TaskGraph graph;
+ bool has_tasks_required_for_activation = false;
+ for (RasterTaskVector::const_iterator it = raster_tasks().begin();
+ it != raster_tasks().end();
+ ++it) {
+ if (IsRasterTaskRequiredForActivation(it->get())) {
+ has_tasks_required_for_activation = true;
+ break;
+ }
+ }
+
scoped_refptr<internal::WorkerPoolTask>
new_raster_required_for_activation_finished_task(
- CreateRasterRequiredForActivationFinishedTask());
+ CreateRasterRequiredForActivationFinishedTask(
+ has_tasks_required_for_activation));
reveman 2013/12/19 17:29:29 Can you use raster_tasks_required_for_activation()
internal::GraphNode* raster_required_for_activation_finished_node =
CreateGraphNodeForTask(
new_raster_required_for_activation_finished_task.get(),
« no previous file with comments | « no previous file | cc/resources/pixel_buffer_raster_worker_pool.cc » ('j') | cc/resources/raster_worker_pool.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698