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

Unified Diff: cc/resources/image_raster_worker_pool.cc

Issue 21159007: cc: Adding support for RGBA_4444 tile textures (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix raster-on-demand codepath Created 7 years, 3 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/image_raster_worker_pool.cc
diff --git a/cc/resources/image_raster_worker_pool.cc b/cc/resources/image_raster_worker_pool.cc
index 9862484ccf0aa4945cd94886650505bf47152173..5284f2dfdaab5cbe35a579e39a1040067f82743b 100644
--- a/cc/resources/image_raster_worker_pool.cc
+++ b/cc/resources/image_raster_worker_pool.cc
@@ -34,14 +34,7 @@ class ImageWorkerPoolTaskImpl : public internal::WorkerPoolTask {
if (!buffer_)
return;
- SkBitmap bitmap;
- bitmap.setConfig(SkBitmap::kARGB_8888_Config,
- task_->resource()->size().width(),
- task_->resource()->size().height(),
- stride_);
- bitmap.setPixels(buffer_);
- SkBitmapDevice device(bitmap);
- task_->RunOnWorkerThread(&device, thread_index);
+ task_->RunOnWorkerThread(thread_index, buffer_, task_->resource()->size());
Sami 2013/09/05 14:13:26 Any reason not to pass the bitmap into RunOnWorker
reveman 2013/09/05 16:24:17 you're no longer respecting the value of |stride_|
kaanb 2013/09/06 02:05:54 Done.
kaanb 2013/09/06 02:05:54 Basically it is to ensure the result of copyTo end
}
virtual void CompleteOnOriginThread() OVERRIDE {
reply_.Run(!HasFinishedRunning());

Powered by Google App Engine
This is Rietveld 408576698