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

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: 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « cc/raster_worker_pool.h ('k') | cc/tile_manager.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 WillRunOnThread(base::Thread* thread) OVERRIDE {
25 picture_pile_ = picture_pile_->GetCloneForDrawingOnThread(thread);
26 }
27
24 virtual void Run(RenderingStats* rendering_stats) OVERRIDE { 28 virtual void Run(RenderingStats* rendering_stats) OVERRIDE {
25 task_.Run(picture_pile_.get(), rendering_stats); 29 task_.Run(picture_pile_.get(), rendering_stats);
26 base::subtle::Release_Store(&completed_, 1); 30 base::subtle::Release_Store(&completed_, 1);
27 } 31 }
28 32
29 private: 33 private:
30 scoped_refptr<PicturePileImpl> picture_pile_; 34 scoped_refptr<PicturePileImpl> picture_pile_;
31 RasterWorkerPool::RasterCallback task_; 35 RasterWorkerPool::RasterCallback task_;
32 }; 36 };
33 37
34 } // namespace 38 } // namespace
35 39
36 RasterWorkerPool::RasterWorkerPool( 40 RasterWorkerPool::RasterWorkerPool(
37 WorkerPoolClient* client, size_t num_threads) 41 WorkerPoolClient* client, size_t num_threads)
38 : WorkerPool(client, num_threads) { 42 : WorkerPool(client, num_threads) {
39 } 43 }
40 44
41 RasterWorkerPool::~RasterWorkerPool() { 45 RasterWorkerPool::~RasterWorkerPool() {
42 } 46 }
43 47
44 void RasterWorkerPool::PostRasterTaskAndReply(PicturePileImpl* picture_pile, 48 void RasterWorkerPool::PostRasterTaskAndReply(PicturePileImpl* picture_pile,
49 bool is_cheap,
45 const RasterCallback& task, 50 const RasterCallback& task,
46 const base::Closure& reply) { 51 const base::Closure& reply) {
47 Worker* worker = GetWorkerForNextTask(); 52 PostTask(
48
49 scoped_refptr<PicturePileImpl> picture_pile_clone =
50 picture_pile->GetCloneForDrawingOnThread(worker);
51
52 worker->PostTask(
53 make_scoped_ptr(new RasterWorkerPoolTaskImpl( 53 make_scoped_ptr(new RasterWorkerPoolTaskImpl(
54 picture_pile_clone.get(), 54 picture_pile,
55 task, 55 task,
56 reply)).PassAs<internal::WorkerPoolTask>()); 56 reply)).PassAs<internal::WorkerPoolTask>(),
57 is_cheap);
57 } 58 }
58 59
59 } // namespace cc 60 } // namespace cc
OLDNEW
« 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