Reland MD Audio Player.

Reland MD Audio Player.

This CL includes following patches.
crrev.com/1415953006 Fix accessibility issues in AudioPlayer.
crrev.com/1413713013 AudioPlayer: Update header design.
crrev.com/1416373006 Read Polymer elements' properties directly in closure compilation of Files/AudioPlayer.
crrev.com/1431523003 AudioPlayer: Start playing a track when play icon on the track list is clicked.
crrev.com/1403303011 AudioPlayer: Update control panel with playback progress bar.
crrev.com/1410093005 AudioPlayer: Update color, font, and playlist.
crrev.com/1415923008 AudioPlayer: Update buttons on control panel.
crrev.com/1417983003 AudioPlayer: Update app icons.

BUG=488229
TBR=mtomasz@chromium.org
TEST=manually

Committed: https://crrev.com/76b338aba706ccc03d2e62225f9e64a58d5a8a0a
Cr-Commit-Position: refs/heads/master@{#360503}

[fukino, 2015-11-18 03:07:03]

Description was changed from

==========
Reland MD Audio Player.

BUG=488229
TEST=manually
==========

to

==========
Reland MD Audio Player.

This CL includes following patches.
crrev.com/1415953006 Fix accessibility issues in AudioPlayer.
crrev.com/1413713013 AudioPlayer: Update header design.
crrev.com/1416373006 Read Polymer elements' properties directly in closure
compilation of Files/AudioPlayer.
crrev.com/1431523003 AudioPlayer: Start playing a track when play icon on the
track list is clicked.
crrev.com/1403303011 AudioPlayer: Update control panel with playback progress
bar.
crrev.com/1410093005 AudioPlayer: Update color, font, and playlist.
crrev.com/1415923008 AudioPlayer: Update buttons on control panel.
crrev.com/1417983003 AudioPlayer: Update app icons.

BUG=488229
TEST=manually
==========

[fukino, 2015-11-18 03:07:51]

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

[fukino, 2015-11-18 03:07:51]

Awano-san, could you take a look?
For volume slider, the spec has changed, so it is not included in this CL.
Thanks!

[yawano, 2015-11-18 18:17:29]

lgtm. Thank you!

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

https://codereview.chromium.org/1458733003/diff/1/ui/file_manager/audio_playe...
ui/file_manager/audio_player/elements/track_list.js:260: * @param {boolean}
forcePlay True if the track should be played regardless
small optional nit: forcePlay -> forceToPlay.

[fukino, 2015-11-19 02:14:39]

Thank you!

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

https://codereview.chromium.org/1458733003/diff/1/ui/file_manager/audio_playe...
ui/file_manager/audio_player/elements/track_list.js:260: * @param {boolean}
forcePlay True if the track should be played regardless
On 2015/11/18 18:17:29, yawano wrote:
> small optional nit: forcePlay -> forceToPlay.

Done.

[fukino, 2015-11-19 02:15:53]

Description was changed from

==========
Reland MD Audio Player.

This CL includes following patches.
crrev.com/1415953006 Fix accessibility issues in AudioPlayer.
crrev.com/1413713013 AudioPlayer: Update header design.
crrev.com/1416373006 Read Polymer elements' properties directly in closure
compilation of Files/AudioPlayer.
crrev.com/1431523003 AudioPlayer: Start playing a track when play icon on the
track list is clicked.
crrev.com/1403303011 AudioPlayer: Update control panel with playback progress
bar.
crrev.com/1410093005 AudioPlayer: Update color, font, and playlist.
crrev.com/1415923008 AudioPlayer: Update buttons on control panel.
crrev.com/1417983003 AudioPlayer: Update app icons.

BUG=488229
TEST=manually
==========

to

==========
Reland MD Audio Player.

This CL includes following patches.
crrev.com/1415953006 Fix accessibility issues in AudioPlayer.
crrev.com/1413713013 AudioPlayer: Update header design.
crrev.com/1416373006 Read Polymer elements' properties directly in closure
compilation of Files/AudioPlayer.
crrev.com/1431523003 AudioPlayer: Start playing a track when play icon on the
track list is clicked.
crrev.com/1403303011 AudioPlayer: Update control panel with playback progress
bar.
crrev.com/1410093005 AudioPlayer: Update color, font, and playlist.
crrev.com/1415923008 AudioPlayer: Update buttons on control panel.
crrev.com/1417983003 AudioPlayer: Update app icons.

BUG=488229
TBR=mtomasz@chromium.org
TEST=manually
==========

[fukino, 2015-11-19 02:16:29]

TBRing mtomasz@ for
chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc,
since it's already reviewed in https://codereview.chromium.org/1415953006/.

[fukino, 2015-11-19 02:16:39]

The CQ bit was checked by fukino@chromium.org

[fukino, 2015-11-19 02:16:39]

The patchset sent to the CQ was uploaded after l-g-t-m from yawano@chromium.org
Link to the patchset: https://codereview.chromium.org/1458733003/#ps20001
(title: "forcePlay -> forceToPlay")

[commit-bot: I haz the power, 2015-11-19 02:18:08]

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

[commit-bot: I haz the power, 2015-11-19 02:57:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-19 02:58:38]

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