Merge lp:~lool/mediaplayer-app/qtpowerd-keep-screen-on into lp:mediaplayer-app

Proposed by Loïc Minier
Status: Rejected
Rejected by: Renato Araujo Oliveira Filho
Proposed branch: lp:~lool/mediaplayer-app/qtpowerd-keep-screen-on
Merge into: lp:mediaplayer-app
Diff against target: 31 lines (+3/-0)
2 files modified
debian/control (+1/-0)
src/qml/player/AbstractPlayer.qml (+2/-0)
To merge this branch: bzr merge lp:~lool/mediaplayer-app/qtpowerd-keep-screen-on
Reviewer Review Type Date Requested Status
Renato Araujo Oliveira Filho (community) Disapprove
PS Jenkins bot continuous-integration Needs Fixing
Didier Roche-Tolomelli Approve
Review via email: mp+191061@code.launchpad.net

Commit message

Use QtPowerd to keep display on when playing.

Description of the change

Use QtPowerd to keep display on when playing.

Tested with binaries provided by Ricardo Mendoza with https://code.launchpad.net/~ricmm/qtpowerd/use-pid-for-name/+merge/191052 merged.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

working fine on mako with latest powerd

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

this code was moved to media-hub

review: Disapprove

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2013-09-25 23:08:08 +0000
3+++ debian/control 2013-10-14 21:48:36 +0000
4@@ -23,6 +23,7 @@
5 Architecture: i386 amd64 armhf
6 Depends: ${misc:Depends},
7 ${shlibs:Depends},
8+ qtdeclarative5-qtpowerd0.1,
9 qtdeclarative5-qtmultimedia-plugin | libqt5declarative5-qtmultimedia-plugin,
10 qtdeclarative5-ubuntu-ui-toolkit-plugin | qt-components-ubuntu,
11 qtdeclarative5-ubuntu-ui-extras0.1,
12
13=== modified file 'src/qml/player/AbstractPlayer.qml'
14--- src/qml/player/AbstractPlayer.qml 2013-04-18 21:52:51 +0000
15+++ src/qml/player/AbstractPlayer.qml 2013-10-14 21:48:36 +0000
16@@ -20,6 +20,7 @@
17 */
18 import QtQuick 2.0
19 import QtMultimedia 5.0
20+import QtPowerd 0.1
21 import "../common"
22 import "../common/utils.js" as Utils
23
24@@ -27,6 +28,7 @@
25 id: player
26
27 property bool playing: state == "playing"
28+ onPlayingChanged: QtPowerd.keepDisplayOn = playing
29 property bool paused: state == "paused"
30 property real aspectRatio: width / height
31 property variant video: mediaPlayer

Subscribers

People subscribed via source and target branches