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

Unified Diff: cc/raster_worker_pool.cc

Issue 12194015: cc: Rasterize cheap tiles immediately (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Refactoring PostTask. Created 7 years, 10 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 | « cc/raster_worker_pool.h ('k') | cc/tile_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/raster_worker_pool.cc
diff --git a/cc/raster_worker_pool.cc b/cc/raster_worker_pool.cc
index 13113b5f1bfde8bf7289a9d32bad34de8d86ad99..07650934ca49cb4645fcf0a5ce538fbc6fb6f881 100644
--- a/cc/raster_worker_pool.cc
+++ b/cc/raster_worker_pool.cc
@@ -21,6 +21,10 @@ class RasterWorkerPoolTaskImpl : public internal::WorkerPoolTask {
DCHECK(picture_pile_);
}
+ virtual void WillRunOnThread(base::Thread* thread) OVERRIDE {
+ picture_pile_ = picture_pile_->GetCloneForDrawingOnThread(thread);
+ }
+
virtual void Run(RenderingStats* rendering_stats) OVERRIDE {
task_.Run(picture_pile_.get(), rendering_stats);
base::subtle::Release_Store(&completed_, 1);
@@ -42,18 +46,15 @@ RasterWorkerPool::~RasterWorkerPool() {
}
void RasterWorkerPool::PostRasterTaskAndReply(PicturePileImpl* picture_pile,
+ bool is_cheap,
const RasterCallback& task,
const base::Closure& reply) {
- Worker* worker = GetWorkerForNextTask();
-
- scoped_refptr<PicturePileImpl> picture_pile_clone =
- picture_pile->GetCloneForDrawingOnThread(worker);
-
- worker->PostTask(
+ PostTask(
make_scoped_ptr(new RasterWorkerPoolTaskImpl(
- picture_pile_clone.get(),
+ picture_pile,
task,
- reply)).PassAs<internal::WorkerPoolTask>());
+ reply)).PassAs<internal::WorkerPoolTask>(),
+ is_cheap);
}
} // namespace cc
« no previous file with comments | « cc/raster_worker_pool.h ('k') | cc/tile_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698