cc: Remove cheapness estimator usage from tile manager.

cc/resources/raster_worker_pool.h

 // 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.
 
 #ifndef CC_RESOURCES_RASTER_WORKER_POOL_H_
 #define CC_RESOURCES_RASTER_WORKER_POOL_H_
 
 #include <string>
 
 #include "cc/base/worker_pool.h"
 
 namespace cc {
 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();
 
   static scoped_ptr<RasterWorkerPool> Create(
       WorkerPoolClient* client, size_t num_threads) {
     return make_scoped_ptr(new RasterWorkerPool(client, num_threads));
   }
 
   void PostRasterTaskAndReply(PicturePileImpl* picture_pile,
-                              bool is_cheap,
                               const RasterCallback& task,
                               const base::Closure& reply);
 
  private:
   RasterWorkerPool(WorkerPoolClient* client, size_t num_threads);
 
   DISALLOW_COPY_AND_ASSIGN(RasterWorkerPool);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_RASTER_WORKER_POOL_H_