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

Unified Diff: cc/resources/raster_worker_pool_perftest.cc

Issue 137893023: cc_perftests: Don't count number of tasks required for activation twice (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 months 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/raster_worker_pool_perftest.cc
diff --git a/cc/resources/raster_worker_pool_perftest.cc b/cc/resources/raster_worker_pool_perftest.cc
index f22004798fb3f808bea62d7b154cae3a24cdd3e4..1a888a41ba2cb67229a9c13a7501fb0e13d18620 100644
--- a/cc/resources/raster_worker_pool_perftest.cc
+++ b/cc/resources/raster_worker_pool_perftest.cc
@@ -75,17 +75,10 @@ class PerfRasterWorkerPool : public RasterWorkerPool {
unsigned priority = 0;
TaskGraph graph;
- size_t raster_tasks_required_for_activation_count = 0u;
- for (RasterTaskVector::const_iterator it = raster_tasks().begin();
- it != raster_tasks().end(); ++it) {
- if (IsRasterTaskRequiredForActivation(it->get()))
- raster_tasks_required_for_activation_count++;
- }
-
scoped_refptr<internal::WorkerPoolTask>
raster_required_for_activation_finished_task(
CreateRasterRequiredForActivationFinishedTask(
- raster_tasks_required_for_activation_count));
+ raster_tasks_required_for_activation().size()));
internal::GraphNode* raster_required_for_activation_finished_node =
CreateGraphNodeForTask(
raster_required_for_activation_finished_task.get(),
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698