on xenial help sign in
Hook auto brightness to unity screen instead of powerd
commit00dd2417fe35194a31dc1c41a48d6225e4bdf160 authorAlberto Aguirre <alberto.aguirre@canonical.com> dateMon 16 Jun 2014, 22:52 (12 years ago) parent90de529
3 files changed+8−14
plugins/Powerd/Powerd.cpp+6−11 plugins/Powerd/Powerd.h+1−2 tests/mocks/Powerd/Powerd.h+1−1
plugins/Powerd/Powerd.cpp+6−11
@@ -19,21 +19,16 @@
1919#include "Powerd.h"
2020#include <QDBusPendingCall>
2121
22−void autoBrightnessChanged(GSettings *settings, const gchar *key, QDBusInterface *powerd)
22+void autoBrightnessChanged(GSettings *settings, const gchar *key, QDBusInterface *unityScreen)
2323{
2424 bool value = g_settings_get_boolean(settings, key);
25− powerd->asyncCall("userAutobrightnessEnable", QVariant(value));
25+ unityScreen->asyncCall("userAutobrightnessEnable", QVariant(value));
2626}
2727
2828Powerd::Powerd(QObject* parent)
2929 : QObject(parent),
30− powerd(NULL)
30+ unityScreen(NULL)
3131{
32− powerd = new QDBusInterface("com.canonical.powerd",
33− "/com/canonical/powerd",
34− "com.canonical.powerd",
35− QDBusConnection::SM_BUSNAME(), this);
36−
3732 unityScreen = new QDBusInterface("com.canonical.Unity.Screen",
3833 "/com/canonical/Unity/Screen",
3934 "com.canonical.Unity.Screen",
@@ -47,12 +42,12 @@ Powerd::Powerd(QObject* parent)
4742 SIGNAL(displayPowerStateChange(int, int)));
4843
4944 systemSettings = g_settings_new("com.ubuntu.touch.system");
50− g_signal_connect(systemSettings, "changed::auto-brightness", G_CALLBACK(autoBrightnessChanged), powerd);
51− autoBrightnessChanged(systemSettings, "auto-brightness", powerd);
45+ g_signal_connect(systemSettings, "changed::auto-brightness", G_CALLBACK(autoBrightnessChanged), unityScreen);
46+ autoBrightnessChanged(systemSettings, "auto-brightness", unityScreen);
5247}
5348
5449Powerd::~Powerd()
5550{
56− g_signal_handlers_disconnect_by_data(systemSettings, powerd);
51+ g_signal_handlers_disconnect_by_data(systemSettings, unityScreen);
5752 g_object_unref(systemSettings);
5853}
plugins/Powerd/Powerd.h+1−2
@@ -32,7 +32,7 @@ class Powerd: public QObject
3232
3333public:
3434 enum DisplayStateChangeReason {
35− Normal = 0,
35+ Unknown = 0,
3636 Inactivity = 1, // Display changed state due to inactivity
3737 PowerKey = 2, // Display changed state due to user pressing power key
3838 Proximity = 3, // Display changed state due to proximity events
@@ -50,7 +50,6 @@ Q_SIGNALS:
5050 void displayPowerStateChange(int status, int reason);
5151
5252private:
53− QDBusInterface *powerd;
5453 QDBusInterface *unityScreen;
5554 GSettings *systemSettings;
5655};
tests/mocks/Powerd/Powerd.h+1−1
@@ -29,7 +29,7 @@ class Powerd: public QObject
2929
3030public:
3131 enum DisplayStateChangeReason {
32− Normal = 0,
32+ Unknown = 0,
3333 Inactivity = 1, // Display changed state due to inactivity
3434 PowerKey = 2, // Display changed state due to user pressing power key
3535 Proximity = 3, // Display changed state due to proximity events