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

Side by Side Diff: cc/resources/raster_worker_pool.h

Issue 18614012: cc: Fix incorrect completion of tiles and missing "ready to activate" signal. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CC_RESOURCES_RASTER_WORKER_POOL_H_ 5 #ifndef CC_RESOURCES_RASTER_WORKER_POOL_H_
6 #define CC_RESOURCES_RASTER_WORKER_POOL_H_ 6 #define CC_RESOURCES_RASTER_WORKER_POOL_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/containers/hash_tables.h" 10 #include "base/containers/hash_tables.h"
(...skipping 209 matching lines...) Expand 10 before | Expand all | Expand 10 after
220 220
221 void SetRasterTasks(RasterTask::Queue* queue); 221 void SetRasterTasks(RasterTask::Queue* queue);
222 bool IsRasterTaskRequiredForActivation( 222 bool IsRasterTaskRequiredForActivation(
223 internal::RasterWorkerPoolTask* task) const; 223 internal::RasterWorkerPoolTask* task) const;
224 224
225 RasterWorkerPoolClient* client() const { return client_; } 225 RasterWorkerPoolClient* client() const { return client_; }
226 ResourceProvider* resource_provider() const { return resource_provider_; } 226 ResourceProvider* resource_provider() const { return resource_provider_; }
227 const RasterTask::Queue::TaskVector& raster_tasks() const { 227 const RasterTask::Queue::TaskVector& raster_tasks() const {
228 return raster_tasks_; 228 return raster_tasks_;
229 } 229 }
230 const RasterTask::Queue::TaskSet& raster_tasks_required_for_activation()
vmpstr 2013/07/10 16:46:49 This one is also just for a DCHECK, right?
231 const {
232 return raster_tasks_required_for_activation_;
233 }
230 void set_raster_finished_task( 234 void set_raster_finished_task(
231 scoped_refptr<internal::WorkerPoolTask> raster_finished_task) { 235 scoped_refptr<internal::WorkerPoolTask> raster_finished_task) {
232 raster_finished_task_ = raster_finished_task; 236 raster_finished_task_ = raster_finished_task;
233 } 237 }
234 void set_raster_required_for_activation_finished_task( 238 void set_raster_required_for_activation_finished_task(
235 scoped_refptr<internal::WorkerPoolTask> 239 scoped_refptr<internal::WorkerPoolTask>
236 raster_required_for_activation_finished_task) { 240 raster_required_for_activation_finished_task) {
237 raster_required_for_activation_finished_task_ = 241 raster_required_for_activation_finished_task_ =
238 raster_required_for_activation_finished_task; 242 raster_required_for_activation_finished_task;
239 } 243 }
(...skipping 27 matching lines...) Expand all
267 271
268 base::WeakPtrFactory<RasterWorkerPool> weak_ptr_factory_; 272 base::WeakPtrFactory<RasterWorkerPool> weak_ptr_factory_;
269 scoped_refptr<internal::WorkerPoolTask> raster_finished_task_; 273 scoped_refptr<internal::WorkerPoolTask> raster_finished_task_;
270 scoped_refptr<internal::WorkerPoolTask> 274 scoped_refptr<internal::WorkerPoolTask>
271 raster_required_for_activation_finished_task_; 275 raster_required_for_activation_finished_task_;
272 }; 276 };
273 277
274 } // namespace cc 278 } // namespace cc
275 279
276 #endif // CC_RESOURCES_RASTER_WORKER_POOL_H_ 280 #endif // CC_RESOURCES_RASTER_WORKER_POOL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698