Set max disk cache size correctly.

Set max disk cache size correctly.

Currently we set the max shader cache size too late. The disk cache will
be created with a cache size of 0, and the max_cache_size will have no
effect. This CL changes the default from 0 to the desired
gpu::kDefaultMaxProgramCacheMemoryBytes so we always create the disk cache
with the correct size to match the memory cache (currently set to 6MiB).

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

[dsinclair, 2013-06-17 14:55:09]

apatrick, PTAL.

[pasko, 2013-06-17 16:47:28]

lgtm, thanks!

thinking aloud: 6MB would be scary for low-end mobile devices, but may actually
be cute for a beast like Nexus 10. Dynamically checking this scares me off,
however.

[pasko, 2013-06-17 16:47:37]

https://codereview.chromium.org/17283002/diff/1/content/browser/gpu/shader_di...
File content/browser/gpu/shader_disk_cache.cc (right):

https://codereview.chromium.org/17283002/diff/1/content/browser/gpu/shader_di...
content/browser/gpu/shader_disk_cache.cc:506:
max_cache_size_(gpu::kDefaultMaxProgramCacheMemoryBytes),
The field is used only once, directly feeding the constant to backend
initialization would be even simpler.

[apatrick_chromium, 2013-06-17 18:22:43]

LGTM.

[dsinclair, 2013-06-17 18:59:10]

https://codereview.chromium.org/17283002/diff/1/content/browser/gpu/shader_di...
File content/browser/gpu/shader_disk_cache.cc (right):

https://codereview.chromium.org/17283002/diff/1/content/browser/gpu/shader_di...
content/browser/gpu/shader_disk_cache.cc:506:
max_cache_size_(gpu::kDefaultMaxProgramCacheMemoryBytes),
On 2013/06/17 16:47:37, pasko wrote:
> The field is used only once, directly feeding the constant to backend
> initialization would be even simpler.

Done.

[commit-bot: I haz the power, 2013-06-17 19:51:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dsinclair@chromium.org/17283002/4002

[commit-bot: I haz the power, 2013-06-17 23:59:30]

Change committed as 206849