-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109846/
-----------------------------------------------------------
(Updated April 6, 2013, 4:13 p.m.)
Status
------
This change has been marked as submitted.
Review request for Amarok and Oliver Henshaw.
Description
-------
1. Added "Suspend Options" to the PlaybackConfig Dialog, with two options:
Pause playback on suspend
Inhibit suspend if playing
2. Created a PowerManager class to handle suspend behavior
Fixes:
BR 259862 - Amarok does not inhibit suspend while playing
BR 222571 - Amarok pause on suspend
Diffs
-----
src/playback/PowerManager.h PRE-CREATION
src/configdialog/dialogs/PlaybackConfig.ui 3a79e43
src/amarokconfig.kcfg fbe5497
src/EngineController.cpp 52bfd90
ChangeLog 3422cad
src/CMakeLists.txt 990f313
src/playback/PowerManager.cpp PRE-CREATION
tests/core-impl/meta/multi/CMakeLists.txt 327b555
tests/playlist/CMakeLists.txt c2e4c06
Diff: http://git.reviewboard.kde.org/r/109846/diff/
Testing
-------
Works as expected
File Attachments
----------------
PlaybackConfig Dialog
http://git.reviewboard.kde.org/media/uploaded/files/2013/04/03/snapshot11.png
Thanks,
Anmol Ahuja
_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel
