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

Side by Side 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: Schedule cheap tasks in worker pool. 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 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 #include "cc/raster_worker_pool.h" 5 #include "cc/raster_worker_pool.h"
6 6
7 #include "cc/picture_pile_impl.h" 7 #include "cc/picture_pile_impl.h"
8 8
9 namespace cc { 9 namespace cc {
10 10
11 namespace { 11 namespace {
12 12
13 class RasterWorkerPoolTaskImpl : public internal::WorkerPoolTask { 13 class RasterWorkerPoolTaskImpl : public internal::WorkerPoolTask {
14 public: 14 public:
15 RasterWorkerPoolTaskImpl(PicturePileImpl* picture_pile, 15 RasterWorkerPoolTaskImpl(PicturePileImpl* picture_pile,
16 const RasterWorkerPool::RasterCallback& task, 16 const RasterWorkerPool::RasterCallback& task,
17 const base::Closure& reply) 17 const base::Closure& reply)
18 : internal::WorkerPoolTask(reply), 18 : internal::WorkerPoolTask(reply),
19 picture_pile_(picture_pile), 19 picture_pile_(picture_pile),
20 task_(task) { 20 task_(task) {
21 DCHECK(picture_pile_); 21 DCHECK(picture_pile_);
22 } 22 }
23 23
24 virtual void Run(RenderingStats* rendering_stats) OVERRIDE { 24 virtual void Run(RenderingStats* rendering_stats) OVERRIDE {
25 task_.Run(picture_pile_.get(), rendering_stats); 25 task_.Run(picture_pile_.get(), rendering_stats);
26 } 26 }
27 27
28 virtual void DeferToThread(base::Thread* thread) OVERRIDE {
29 picture_pile_ = picture_pile_->GetCloneForDrawingOnThread(thread);
reveman 2013/02/13 22:17:36 If we need this, I prefer if we didn't also do clo
30 }
31
28 private: 32 private:
29 scoped_refptr<PicturePileImpl> picture_pile_; 33 scoped_refptr<PicturePileImpl> picture_pile_;
30 RasterWorkerPool::RasterCallback task_; 34 RasterWorkerPool::RasterCallback task_;
31 }; 35 };
32 36
33 } // namespace 37 } // namespace
34 38
35 RasterWorkerPool::RasterWorkerPool(size_t num_threads) 39 RasterWorkerPool::RasterWorkerPool(size_t num_threads)
36 : WorkerPool(num_threads) { 40 : WorkerPool(num_threads) {
37 } 41 }
38 42
39 RasterWorkerPool::~RasterWorkerPool() { 43 RasterWorkerPool::~RasterWorkerPool() {
40 } 44 }
41 45
42 void RasterWorkerPool::PostRasterTaskAndReply(PicturePileImpl* picture_pile, 46 void RasterWorkerPool::PostRasterTaskAndReply(PicturePileImpl* picture_pile,
47 bool is_cheap,
43 const RasterCallback& task, 48 const RasterCallback& task,
44 const base::Closure& reply) { 49 const base::Closure& reply) {
50 if (is_cheap && CanPostCheapTask()) {
51 PostCheapTask(
52 make_scoped_ptr(new RasterWorkerPoolTaskImpl(
53 picture_pile,
54 task,
55 reply)).PassAs<internal::WorkerPoolTask>());
56 return;
57 }
58
45 Worker* worker = GetWorkerForNextTask(); 59 Worker* worker = GetWorkerForNextTask();
46 60
47 scoped_refptr<PicturePileImpl> picture_pile_clone = 61 scoped_refptr<PicturePileImpl> picture_pile_clone =
48 picture_pile->GetCloneForDrawingOnThread(worker); 62 picture_pile->GetCloneForDrawingOnThread(worker);
49 63
50 worker->PostTask( 64 worker->PostTask(
51 make_scoped_ptr(new RasterWorkerPoolTaskImpl( 65 make_scoped_ptr(new RasterWorkerPoolTaskImpl(
52 picture_pile_clone.get(), 66 picture_pile_clone.get(),
53 task, 67 task,
54 reply)).PassAs<internal::WorkerPoolTask>()); 68 reply)).PassAs<internal::WorkerPoolTask>());
55 } 69 }
56 70
57 } // namespace cc 71 } // namespace cc
OLDNEW
« no previous file with comments | « cc/raster_worker_pool.h ('k') | cc/tile_manager.h » ('j') | cc/tile_manager.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698