Files.app: Start to use new metadata model in Audio Player.

Files.app: Start to use new metadata model in Audio Player.

BUG=410766
TEST=None

Committed: https://crrev.com/395240b4e8e66980b4c1d2d173f29688c54c3ad0
Cr-Commit-Position: refs/heads/master@{#317019}

[hirono, 2015-02-19 07:01:48]

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

[hirono, 2015-02-19 07:01:48]

PTAL, thanks!

[yawano, 2015-02-19 07:51:05]

Thank you!

https://codereview.chromium.org/936243004/diff/40001/ui/file_manager/audio_pl...
File ui/file_manager/audio_player/js/audio_player_scripts.js (right):

https://codereview.chromium.org/936243004/diff/40001/ui/file_manager/audio_pl...
ui/file_manager/audio_player/js/audio_player_scripts.js:27: <include
src="../../file_manager/foreground/js/metadata/metadata_cache_set.js">
Please also add these files to audio_player.html.

https://codereview.chromium.org/936243004/diff/40001/ui/file_manager/audio_pl...
ui/file_manager/audio_player/js/audio_player_scripts.js:40: <include
src="../../file_manager/foreground/js/metadata/metadata_cache.js">
MetadataCache seems not to be used by the js scripts of audio player now. Can't
we remove it now?

[hirono, 2015-02-19 08:36:31]

https://codereview.chromium.org/936243004/diff/40001/ui/file_manager/audio_pl...
File ui/file_manager/audio_player/js/audio_player_scripts.js (right):

https://codereview.chromium.org/936243004/diff/40001/ui/file_manager/audio_pl...
ui/file_manager/audio_player/js/audio_player_scripts.js:27: <include
src="../../file_manager/foreground/js/metadata/metadata_cache_set.js">
On 2015/02/19 07:51:05, yawano wrote:
> Please also add these files to audio_player.html.

Done.

https://codereview.chromium.org/936243004/diff/40001/ui/file_manager/audio_pl...
ui/file_manager/audio_player/js/audio_player_scripts.js:40: <include
src="../../file_manager/foreground/js/metadata/metadata_cache.js">
On 2015/02/19 07:51:05, yawano wrote:
> MetadataCache seems not to be used by the js scripts of audio player now.
Can't
> we remove it now?

Done.

[yawano, 2015-02-19 08:38:10]

lgtm. Thank you!

[hirono, 2015-02-19 08:58:53]

New patchsets have been uploaded after l-g-t-m from yawano@chromium.org

[hirono, 2015-02-19 08:59:14]

The CQ bit was checked by hirono@chromium.org

[hirono, 2015-02-19 08:59:20]

On 2015/02/19 08:38:10, yawano wrote:
> lgtm. Thank you!

Thanks!

[commit-bot: I haz the power, 2015-02-19 08:59:33]

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

[commit-bot: I haz the power, 2015-02-19 09:21:44]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-02-19 09:22:12]

Patchset 6 (id:??) landed as
https://crrev.com/395240b4e8e66980b4c1d2d173f29688c54c3ad0
Cr-Commit-Position: refs/heads/master@{#317019}