Merge lp:~tiagosh/telepathy-ofono/wake-up-screen into lp:telepathy-ofono

Proposed by Tiago Salem Herrmann
Status: Merged
Approved by: Bill Filler
Approved revision: 124
Merged at revision: 123
Proposed branch: lp:~tiagosh/telepathy-ofono/wake-up-screen
Merge into: lp:telepathy-ofono
Diff against target: 34 lines (+13/-2)
1 file modified
powerdaudiomodemediator.cpp (+13/-2)
To merge this branch: bzr merge lp:~tiagosh/telepathy-ofono/wake-up-screen
Reviewer Review Type Date Requested Status
Bill Filler (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+244785@code.launchpad.net

Commit message

Power the screen on before disabling the proximity sensor.

Description of the change

Power the screen on before disabling the proximity sensor.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'powerdaudiomodemediator.cpp'
--- powerdaudiomodemediator.cpp 2014-12-04 13:31:28 +0000
+++ powerdaudiomodemediator.cpp 2014-12-15 19:46:38 +0000
@@ -16,6 +16,7 @@
16 * Authors: Andreas Pokorny <andreas.pokorny@canonical.com>16 * Authors: Andreas Pokorny <andreas.pokorny@canonical.com>
17 */17 */
1818
19#include <QDBusInterface>
19#include "powerdaudiomodemediator.h"20#include "powerdaudiomodemediator.h"
2021
21PowerDAudioModeMediator::PowerDAudioModeMediator(PowerD &powerd)22PowerDAudioModeMediator::PowerDAudioModeMediator(PowerD &powerd)
@@ -36,10 +37,20 @@
3637
37void PowerDAudioModeMediator::apply() const38void PowerDAudioModeMediator::apply() const
38{39{
39 if (mProximityEnabled)40 if (mProximityEnabled) {
40 powerd.enableProximityHandling();41 powerd.enableProximityHandling();
41 else42 } else {
43 // we need to power the screen on before disabling the proximity handling
44 QDBusInterface unityIface("com.canonical.Unity.Screen",
45 "/com/canonical/Unity/Screen",
46 "com.canonical.Unity.Screen",
47 QDBusConnection::systemBus());
48 QList<QVariant> args;
49 args.append("on");
50 args.append(3);
51 unityIface.callWithArgumentList(QDBus::NoBlock, "setScreenPowerMode", args);
42 powerd.disableProximityHandling();52 powerd.disableProximityHandling();
53 }
43}54}
4455
45void PowerDAudioModeMediator::audioOutputClosed()56void PowerDAudioModeMediator::audioOutputClosed()

Subscribers

People subscribed via source and target branches