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

Harald Sitter sitter at kde.org
Wed Sep 4 12:44:20 BST 2013


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


it would be good if you actually submitted the diff for review rather than appending it as attachment.

TIA

- Harald Sitter


On Aug. 30, 2013, 11:05 a.m., James Smith wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112076/
> -----------------------------------------------------------
> 
> (Updated Aug. 30, 2013, 11:05 a.m.)
> 
> 
> Review request for KDE Multimedia and Harald Sitter.
> 
> 
> 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
> -----
> 
>   src/app/mainWindow.h 51ef72c 
>   src/app/mainWindow.cpp f850820 
>   src/app/stateChange.cpp 0edde72 
> 
> Diff: http://git.reviewboard.kde.org/r/112076/diff/
> 
> 
> Testing
> -------
> 
> Compile, run-test
> 
> 
> File Attachments
> ----------------
> 
> stateChange and mainWindow-less power patch
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/08/30/dragoninhibitreworknomainwindow.diff
> 
> 
> Thanks,
> 
> James Smith
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20130904/a6ca2e00/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