Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(229)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/file_manager/audio_player/elements/control_panel.js

Issue 2305623003: Support to repeat one song in Audio Player. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« ui/file_manager/audio_player/elements/compiled_resources2.gyp ('K') | « ui/file_manager/audio_player/elements/control_panel.html ('k') | ui/file_manager/audio_player/elements/repeat_button.css » ('j') | ui/file_manager/audio_player/elements/repeat_button.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/audio_player/elements/control_panel.js
diff --git a/ui/file_manager/audio_player/elements/control_panel.js b/ui/file_manager/audio_player/elements/control_panel.js
index ff36640ab5a061539848d8a9181ae1822f7c86c7..9b139a78ee5c8c3cbbfc9876112fca58e7245a8a 100644
--- a/ui/file_manager/audio_player/elements/control_panel.js
+++ b/ui/file_manager/audio_player/elements/control_panel.js
@@ -77,11 +77,12 @@
},
/**
- * Whether the repeat button is ON.
+ * What mode the repeat button idicates.
+ * repeat-modes can be "repeat-none", "repeat-all", "repeat-one".
*/
repeat: {
fukino 2016/09/02 12:11:48 This property should be repeatMode for consistency
harukam 2016/09/05 04:35:03 Agree. Thanks.
- type: Boolean,
- value: false,
+ type: String,
+ value: "repeat-none",
notify: true
},
« ui/file_manager/audio_player/elements/compiled_resources2.gyp ('K') | « ui/file_manager/audio_player/elements/control_panel.html ('k') | ui/file_manager/audio_player/elements/repeat_button.css » ('j') | ui/file_manager/audio_player/elements/repeat_button.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698