cc: Adding support for RGBA_4444 tile textures

cc/resources/image_raster_worker_pool.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/resources/image_raster_worker_pool.h"
 
 #include "base/debug/trace_event.h"
 #include "base/values.h"
 #include "cc/debug/traced_value.h"
 #include "cc/resources/resource.h"
@@ skipping 16 matching lines @@
         stride_(stride),
         reply_(reply) {
   }
 
   // Overridden from internal::WorkerPoolTask:
   virtual void RunOnWorkerThread(unsigned thread_index) OVERRIDE {
     TRACE_EVENT0("cc", "ImageWorkerPoolTaskImpl::RunOnWorkerThread");
     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 RunOnWorkerThread instead of the raw
buffer? I find passing void* pointers around like this a little scary.

[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 ends up in transfer buffer
memory.

   }
   virtual void CompleteOnOriginThread() OVERRIDE {
     reply_.Run(!HasFinishedRunning());
   }
 
  private:
   virtual ~ImageWorkerPoolTaskImpl() {}
 
   scoped_refptr<internal::RasterWorkerPoolTask> task_;
   uint8_t* buffer_;
@@ skipping 166 matching lines @@
   if (is_required_for_activation) {
     raster_required_for_activation_finished_node->add_dependency();
     image_node->add_dependent(raster_required_for_activation_finished_node);
   }
 
   raster_finished_node->add_dependency();
   image_node->add_dependent(raster_finished_node);
 }
 
 }  // namespace cc