Add MetadataCacheSetStorage implementation for LRUCache.

Files.app: Add MetadataCacheSetStorage implementation for LRUCache.

BUG=410766
TEST=None
R=yawano@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/f2bf2b08efce4cea5bae6e8c24e861cb4056d02d

[hirono, 2015-02-03 01:30:50]

hirono@chromium.org changed reviewers:
+ yawano@chromium.org

[hirono, 2015-02-03 01:30:50]

PTAL? Thanks!

[yawano, 2015-02-03 01:36:45]

lgtm with nit.

https://codereview.chromium.org/895723003/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/js/metadata/metadata_cache_set.js
(right):

https://codereview.chromium.org/895723003/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/metadata/metadata_cache_set.js:199: *
@param {!LRUCache<MetadataCacheItem>} cache LRUCache.
nit: MetadataCacheItem -> !MetadataCacheItem ?

[hirono, 2015-02-03 01:39:44]

Thanks!

https://codereview.chromium.org/895723003/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/js/metadata/metadata_cache_set.js
(right):

https://codereview.chromium.org/895723003/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/metadata/metadata_cache_set.js:199: *
@param {!LRUCache<MetadataCacheItem>} cache LRUCache.
On 2015/02/03 01:36:44, yawano wrote:
> nit: MetadataCacheItem -> !MetadataCacheItem ?

Yes, you are right!

[hirono, 2015-02-03 01:39:47]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-02-03 01:40:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/895723003/20001

[commit-bot: I haz the power, 2015-02-03 03:39:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-03 03:39:42]

Try jobs failed on following builders:
  win8_chromium_rel on tryserver.chromium.win (None)

[hirono, 2015-02-03 03:49:29]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-02-03 03:50:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/895723003/40001

[commit-bot: I haz the power, 2015-02-03 05:19:10]

Patchset 3 (id:??) landed as
https://crrev.com/f2bf2b08efce4cea5bae6e8c24e861cb4056d02d
Cr-Commit-Position: refs/heads/master@{#314271}

[hirono, 2015-02-03 05:19:19]

Committed patchset #3 (id:40001) manually as
f2bf2b08efce4cea5bae6e8c24e861cb4056d02d (presubmit successful).