Files.app: Fix exception thrown by ContentProvider if getLatest is called with the type 'media|thum…

Files.app: Fix exception thrown by ContentProvider if getLatest is called with the type 'media|thumbnail'.

Previously there is assumption in ContentProvider#fetch that only one request is
called for one URL at the same time, and the next request is never called before
the previous request completes. But actually multiple requests can be called for
each meatadata type. ContentProvider handles 'media' and 'thumbnail' metadata
type, so if both metadata type is specified, it causes an exception in
ContentProvider.

BUG=446396
TEST=FileManagerJsTest.MetadataCache

Committed: https://crrev.com/a2fac13bb5fcc690f8d8e8d81a680c1b85b523c0
Cr-Commit-Position: refs/heads/master@{#310061}

[hirono, 2015-01-06 05:44:04]

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

[hirono, 2015-01-06 05:44:05]

PTAL the CL? Thanks!

[yawano, 2015-01-06 06:04:52]

lgtm with very small nit. Thank you!

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

https://codereview.chromium.org/806853007/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js:956: *
@type {!Object<!string, !Array<!Function>>}
very small nit: Can we write this type as {!Object<!string,
!Array<function(Object)>>}?

[hirono, 2015-01-06 06:16:20]

Thanks!

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

https://codereview.chromium.org/806853007/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js:956: *
@type {!Object<!string, !Array<!Function>>}
On 2015/01/06 06:04:52, yawano wrote:
> very small nit: Can we write this type as {!Object<!string,
> !Array<function(Object)>>}?

Yes, done!

[hirono, 2015-01-06 06:16:27]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-01-06 06:17:26]

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

[commit-bot: I haz the power, 2015-01-06 06:59:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-06 07:00:01]

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