cc: Remove cheapness estimator usage from tile manager.

cc/resources/raster_worker_pool.h

Index: cc/resources/raster_worker_pool.h
diff --git a/cc/resources/raster_worker_pool.h b/cc/resources/raster_worker_pool.h
index 055794626269af0469ad307db549db2f9746e35a..8d3691d8ef534b49453907fdbd4623c691c60e82 100644
--- a/cc/resources/raster_worker_pool.h
+++ b/cc/resources/raster_worker_pool.h
@@ -15,8 +15,7 @@ class PicturePileImpl;
 // A worker thread pool that runs raster tasks.
 class RasterWorkerPool : public WorkerPool {
  public:
-  typedef base::Callback<void(PicturePileImpl*)>
-      RasterCallback;
+  typedef base::Callback<void(PicturePileImpl* picture_pile)> RasterCallback;

[vmpstr, 2013/04/26 21:31:55]

That kind of looks weird to me. Is that in line with coding style?

[reveman, 2013/04/26 22:28:25]

this is very inconsistent throughout chromium code. no name seems a bit more
common but I added it as presubmit complains without it. I'll remove it from
this patch as cc/ seem to use no name and I prefer to be consistent. we can
change all of cc/ later if this is wrong.

[vmpstr, 2013/04/26 22:51:35]

Oh I think presubmit might be confusing (PicturePileImpl*) with a C-style cast.
I'm OK with this fix.

 
   virtual ~RasterWorkerPool();
 
@@ -26,7 +25,6 @@ class RasterWorkerPool : public WorkerPool {
   }
 
   void PostRasterTaskAndReply(PicturePileImpl* picture_pile,
-                              bool is_cheap,
                               const RasterCallback& task,
                               const base::Closure& reply);