cc: Made use color estimator a const instead of a flag.

cc: Made use color estimator a const instead of a flag.

Color estimator has been running for some time now, and it seems
to be stable. This change removes the piping of the flag down to
where it's used. The flag is still in the code for easy debugging.

R=reveman@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=209568

[vmpstr, 2013-07-01 18:14:36]

Please take a look.

[reveman, 2013-07-01 20:00:59]

lgtm

[vmpstr, 2013-07-01 20:05:34]

+piman for chrome/browser/chromeos/login/ and content/ OWNER's review. Please
take a look.

[piman, 2013-07-01 20:14:45]

lgtm

https://codereview.chromium.org/18370002/diff/1/cc/resources/raster_worker_po...
File cc/resources/raster_worker_pool.cc (right):

https://codereview.chromium.org/18370002/diff/1/cc/resources/raster_worker_po...
cc/resources/raster_worker_pool.cc:22: const bool kUseColorEstimator = true;
nit: do we even need this?

[vmpstr, 2013-07-01 20:22:18]

https://codereview.chromium.org/18370002/diff/1/cc/resources/raster_worker_po...
File cc/resources/raster_worker_pool.cc (right):

https://codereview.chromium.org/18370002/diff/1/cc/resources/raster_worker_po...
cc/resources/raster_worker_pool.cc:22: const bool kUseColorEstimator = true;
On 2013/07/01 20:14:45, piman wrote:
> nit: do we even need this?

I think it's still useful to disable the color estimator while debugging things
to force everything to do a texture upload. I'd prefer to keep it, instead of
remembering to set analysis_.is_solid_color to false around line 100...

[commit-bot: I haz the power, 2013-07-01 20:38:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vmpstr@chromium.org/18370002/1

[commit-bot: I haz the power, 2013-07-01 20:58:21]

Sorry for I got bad news for ya.
Compile failed with a clobber build on android_dbg.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-07-01 21:32:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vmpstr@chromium.org/18370002/21001

[commit-bot: I haz the power, 2013-07-02 01:30:00]

Change committed as 209568