Audio Player: Notify track metadata update to track-list element explicitly.

Audio Player: Notify track metadata update to track-list element explicitly.

To re-render the track-list element, the metadata change needs to be notified to
the track-list element explicitly.

BUG=575151
TEST=manual test

Committed: https://crrev.com/69b3f0b75b10ef9766e34140dc648e6aceaefd6d
Cr-Commit-Position: refs/heads/master@{#369393}

[fukino, 2016-01-13 05:33:06]

fukino@chromium.org changed reviewers:
+ yoshiki@chromium.org

[fukino, 2016-01-13 05:33:06]

PTAL. Thanks!

[fukino, 2016-01-14 06:08:43]

Ping. > Yoshiki-san

[yoshiki, 2016-01-14 11:10:27]

lgtm with nit

https://codereview.chromium.org/1581573004/diff/1/ui/file_manager/audio_playe...
File ui/file_manager/audio_player/elements/audio_player.js (right):

https://codereview.chromium.org/1581573004/diff/1/ui/file_manager/audio_playe...
ui/file_manager/audio_player/elements/audio_player.js:356:
notifyTrackMetadataUpdated: function(index) {
nit: should we check if the argument is valid just in case?

[fukino, 2016-01-14 11:32:35]

Thank you!

https://codereview.chromium.org/1581573004/diff/1/ui/file_manager/audio_playe...
File ui/file_manager/audio_player/elements/audio_player.js (right):

https://codereview.chromium.org/1581573004/diff/1/ui/file_manager/audio_playe...
ui/file_manager/audio_player/elements/audio_player.js:356:
notifyTrackMetadataUpdated: function(index) {
On 2016/01/14 11:10:27, yoshiki wrote:
> nit: should we check if the argument is valid just in case? 

Added a check for index range. Done.

[fukino, 2016-01-14 11:33:15]

The CQ bit was checked by fukino@chromium.org

[fukino, 2016-01-14 11:33:15]

The patchset sent to the CQ was uploaded after l-g-t-m from yoshiki@chromium.org
Link to the patchset: https://codereview.chromium.org/1581573004/#ps20001
(title: "Add a check for index range.")

[commit-bot: I haz the power, 2016-01-14 11:33:31]

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

[commit-bot: I haz the power, 2016-01-14 11:56:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-14 11:58:35]

Description was changed from

==========
Audio Player: Notify track metadata update to track-list element explicitly.

To re-render the track-list element, the metadata change needs to be notified to
the track-list element explicitly.

BUG=575151
TEST=manual test
==========

to

==========
Audio Player: Notify track metadata update to track-list element explicitly.

To re-render the track-list element, the metadata change needs to be notified to
the track-list element explicitly.

BUG=575151
TEST=manual test

Committed: https://crrev.com/69b3f0b75b10ef9766e34140dc648e6aceaefd6d
Cr-Commit-Position: refs/heads/master@{#369393}
==========

[commit-bot: I haz the power, 2016-01-14 11:58:36]

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