Audio Player: Cancel scheduled advance if the track has been changed

Audio Player: Cancel scheduled advance if the track has been changed

BUG=348487
TEST=manual

Committed: https://crrev.com/8b1ace1021431de695a50a52537a64922b007620
Cr-Commit-Position: refs/heads/master@{#292910}

[yoshiki, 2014-09-02 06:43:56]

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

[fukino, 2014-09-02 06:59:43]

On 2014/09/02 06:43:56, yoshiki wrote:
> mailto:yoshiki@chromium.org changed reviewers:
> + mailto:fukino@chromium.org

The bug id looks incorrect.

[fukino, 2014-09-02 07:33:08]

Ah, I found the issue id.
It must be 348487. (Please update the descripotion).

Regarding the code, lgtm with a nit.

https://codereview.chromium.org/534493002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/audio_player/elements/audio_player.js (right):

https://codereview.chromium.org/534493002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/audio_player/elements/audio_player.js:274: // If
the track has been changed since the advance is scheduled, do
nit: "was" scheduled?

[yoshiki, 2014-09-02 10:56:09]

Thanks! Fixed and going to commit.

https://codereview.chromium.org/534493002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/audio_player/elements/audio_player.js (right):

https://codereview.chromium.org/534493002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/audio_player/elements/audio_player.js:274: // If
the track has been changed since the advance is scheduled, do
On 2014/09/02 07:33:08, fukino wrote:
> nit: "was" scheduled?

Done.

[yoshiki, 2014-09-02 10:56:14]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-09-02 10:56:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yoshiki@chromium.org/534493002/40001

[commit-bot: I haz the power, 2014-09-02 11:52:19]

Committed patchset #3 (id:40001) as bd3b38d3cfa1ccc19e918eedd1703182827b7ce2

[commit-bot: I haz the power, 2014-09-10 03:18:02]

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