Review Request 112076: Fix bug where monitor won't powersave even with only audio content

James Smith smithjd15 at gmail.com
Mon Aug 26 07:22:58 BST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112076/
-----------------------------------------------------------

(Updated Aug. 26, 2013, 6:22 a.m.)


Review request for KDE Multimedia and Harald Sitter.


Changes
-------

Style fixes and returning original inhibitPowerSave function with hasVideo boolean, and additional operation plus commenting in stateChanged.cpp


Description
-------

Fixes an issue where playing audio content keeps the screen awake. Video content is supposed to, but with the new play URL button, dragon should allow the monitor to turn off.


Diffs (updated)
-----

  src/app/stateChange.cpp 0edde72 
  src/app/mainWindow.cpp f850820 
  src/app/mainWindow.h 51ef72c 

Diff: http://git.reviewboard.kde.org/r/112076/diff/


Testing
-------

Compile, run-test


Thanks,

James Smith

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20130826/b4281858/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-multimedia mailing list
kde-multimedia at kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia


More information about the kde-multimedia mailing list