Chromium Code Reviews| 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()); |