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

Unified Diff: cc/resources/raster_worker_pool.cc

Issue 170783007: Re-land: cc: Cleanup internal::WorkerPoolTaskClient interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
Index: cc/resources/raster_worker_pool.cc
diff --git a/cc/resources/raster_worker_pool.cc b/cc/resources/raster_worker_pool.cc
index 925cf94b28afa01633d109f353bb7bd558c8babb..7df47c833798d51b6c69fbff9f32239203247efe 100644
--- a/cc/resources/raster_worker_pool.cc
+++ b/cc/resources/raster_worker_pool.cc
@@ -112,7 +112,7 @@ class RasterWorkerPoolTaskImpl : public internal::RasterWorkerPoolTask {
virtual void ScheduleOnOriginThread(internal::WorkerPoolTaskClient* client)
OVERRIDE {
DCHECK(!canvas_);
- canvas_ = client->AcquireCanvasForRaster(this);
+ canvas_ = client->AcquireCanvasForRaster(this, resource());
}
virtual void RunOnOriginThread() OVERRIDE {
TRACE_EVENT0("cc", "RasterWorkerPoolTaskImpl::RunOnOriginThread");
@@ -135,7 +135,7 @@ class RasterWorkerPoolTaskImpl : public internal::RasterWorkerPoolTask {
virtual void CompleteOnOriginThread(internal::WorkerPoolTaskClient* client)
OVERRIDE {
canvas_ = NULL;
- client->OnRasterCompleted(this, analysis_);
+ client->ReleaseCanvasForRaster(this, resource());
}
virtual void RunReplyOnOriginThread() OVERRIDE {
DCHECK(!canvas_);
@@ -270,9 +270,7 @@ class ImageDecodeWorkerPoolTaskImpl : public internal::WorkerPoolTask {
Decode();
}
virtual void CompleteOnOriginThread(internal::WorkerPoolTaskClient* client)
- OVERRIDE {
- client->OnImageDecodeCompleted(this);
- }
+ OVERRIDE {}
virtual void RunReplyOnOriginThread() OVERRIDE {
reply_.Run(!HasFinishedRunning());
}

Powered by Google App Engine
This is Rietveld 408576698