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

Unified Diff: cc/resources/tile_manager.cc

Issue 454843002: cc: Do bitmap conversion for RasterBuffer in the worker thread. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make RasterBuffer the public interface used by raster tasks. Created 6 years, 4 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
Index: cc/resources/tile_manager.cc
diff --git a/cc/resources/tile_manager.cc b/cc/resources/tile_manager.cc
index 452d9b45ccfa5e65bdc444bdb3c3f36cc616f429..7e94d9dc423b1bf4ed5aacd4cf6d25b34a367b29 100644
--- a/cc/resources/tile_manager.cc
+++ b/cc/resources/tile_manager.cc
@@ -59,14 +59,14 @@ class RasterTaskImpl : public RasterTask {
analyze_picture_(analyze_picture),
rendering_stats_(rendering_stats),
reply_(reply),
- canvas_(NULL) {}
+ raster_buffer_(NULL) {}
// Overridden from Task:
virtual void RunOnWorkerThread() OVERRIDE {
TRACE_EVENT0("cc", "RasterizerTaskImpl::RunOnWorkerThread");
DCHECK(picture_pile_);
- if (canvas_) {
+ if (raster_buffer_) {
AnalyzeAndRaster(picture_pile_->GetCloneForDrawingOnThread(
RasterWorkerPool::GetPictureCloneIndexForCurrentThread()));
}
@@ -74,25 +74,26 @@ class RasterTaskImpl : public RasterTask {
// Overridden from RasterizerTask:
virtual void ScheduleOnOriginThread(RasterizerTaskClient* client) OVERRIDE {
- DCHECK(!canvas_);
- canvas_ = client->AcquireCanvasForRaster(this);
+ DCHECK(!raster_buffer_);
+ raster_buffer_ = client->AcquireBufferForRaster(this);
}
virtual void CompleteOnOriginThread(RasterizerTaskClient* client) OVERRIDE {
- canvas_ = NULL;
- client->ReleaseCanvasForRaster(this);
+ raster_buffer_ = NULL;
+ client->ReleaseBufferForRaster(this);
}
virtual void RunReplyOnOriginThread() OVERRIDE {
- DCHECK(!canvas_);
+ DCHECK(!raster_buffer_);
reply_.Run(analysis_, !HasFinishedRunning());
}
protected:
- virtual ~RasterTaskImpl() { DCHECK(!canvas_); }
+ virtual ~RasterTaskImpl() { DCHECK(!raster_buffer_); }
private:
void AnalyzeAndRaster(PicturePileImpl* picture_pile) {
DCHECK(picture_pile);
- DCHECK(canvas_);
+ DCHECK(raster_buffer_);
+ DCHECK(raster_buffer_->GetSkCanvas());
if (analyze_picture_) {
Analyze(picture_pile);
@@ -101,6 +102,8 @@ class RasterTaskImpl : public RasterTask {
}
Raster(picture_pile);
+
+ raster_buffer_->Flush();
}
void Analyze(PicturePileImpl* picture_pile) {
@@ -141,7 +144,7 @@ class RasterTaskImpl : public RasterTask {
default:
NOTREACHED();
}
- canvas_->setDrawFilter(draw_filter.get());
+ raster_buffer_->GetSkCanvas()->setDrawFilter(draw_filter.get());
base::TimeDelta prev_rasterize_time =
rendering_stats_->impl_thread_rendering_stats().rasterize_time;
@@ -154,7 +157,7 @@ class RasterTaskImpl : public RasterTask {
tile_resolution_ == HIGH_RESOLUTION ? rendering_stats_ : NULL;
DCHECK(picture_pile);
picture_pile->RasterToBitmap(
- canvas_, content_rect_, contents_scale_, stats);
+ raster_buffer_->GetSkCanvas(), content_rect_, contents_scale_, stats);
if (rendering_stats_->record_rendering_stats()) {
base::TimeDelta current_rasterize_time =
@@ -180,7 +183,7 @@ class RasterTaskImpl : public RasterTask {
bool analyze_picture_;
RenderingStatsInstrumentation* rendering_stats_;
const base::Callback<void(const PicturePileImpl::Analysis&, bool)> reply_;
- SkCanvas* canvas_;
+ RasterBuffer* raster_buffer_;
DISALLOW_COPY_AND_ASSIGN(RasterTaskImpl);
};

Powered by Google App Engine
This is Rietveld 408576698