Changes to address setBrightness interface moving from powerd to unity-system-compositor

4 files changed, 66 insertions, 47 deletions

diff --git a/src/Makefile.am b/src/Makefile.am
index 2461592..3f71b60 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -46,8 +46,8 @@ libindciatorpower_upower_a_LDFLAGS = $(COVERAGE_LDFLAGS)
 libindicatorpower_service_a_SOURCES = \
   ib-brightness-control.c \
   ib-brightness-control.h \
-  ib-brightness-powerd-control.c \
-  ib-brightness-powerd-control.h \
+  ib-brightness-uscreen-control.c \
+  ib-brightness-uscreen-control.h \
   device-provider.c \
   device-provider.h \
   device.c \
diff --git a/src/ib-brightness-powerd-control.c b/src/ib-brightness-uscreen-control.c
index 9e8815b..ad2c155 100644
--- a/src/ib-brightness-powerd-control.c
+++ b/src/ib-brightness-uscreen-control.c
@@ -17,19 +17,20 @@
  *     Yuan-Chen Cheng <yc.cheng@canonical.com>
  */
 
-#include "ib-brightness-powerd-control.h"
+#include "ib-brightness-uscreen-control.h"
 
 static gboolean getBrightnessParams(GDBusProxy* powerd_proxy, int *dim, int *min,
     int *max, int *dflt, gboolean *ab_supported);
 
 GDBusProxy*
-powerd_get_proxy(brightness_params_t *params)
+uscreen_get_proxy(brightness_params_t *params)
 {
     GError *error = NULL;
     gboolean ret;
 
     g_return_val_if_fail (params != NULL, NULL);
 
+    /* For now we still need to obtain the brigthness params from powerd */
     GDBusProxy* powerd_proxy = g_dbus_proxy_new_for_bus_sync(G_BUS_TYPE_SYSTEM,
                 G_DBUS_PROXY_FLAGS_NONE,
                 NULL,
@@ -55,8 +56,26 @@ powerd_get_proxy(brightness_params_t *params)
         g_object_unref (powerd_proxy);
         return NULL;
     }
-    
-    return powerd_proxy;
+
+    g_clear_object (&powerd_proxy);
+
+    GDBusProxy* uscreen_proxy = g_dbus_proxy_new_for_bus_sync(G_BUS_TYPE_SYSTEM,
+                G_DBUS_PROXY_FLAGS_NONE,
+                NULL,
+                "com.canonical.Unity.Screen",
+                "/com/canonical/Unity/Screen",
+                "com.canonical.Unity.Screen",
+                NULL,
+                &error);
+
+    if (error != NULL)
+    {
+        g_debug ("could not connect to unity screen: %s", error->message);
+        g_error_free (error);
+        return NULL;
+    }
+
+    return uscreen_proxy;
 }
 
 
@@ -89,18 +108,18 @@ getBrightnessParams(GDBusProxy* powerd_proxy, int *dim, int *min, int *max, int
     return TRUE;
 }
 
-static gboolean setUserBrightness(GDBusProxy* powerd_proxy, GCancellable *gcancel, int brightness)
+static gboolean setUserBrightness(GDBusProxy* uscreen_proxy, GCancellable *gcancel, int brightness)
 {
     GVariant *ret = NULL;
     GError *error = NULL;
 
-    ret = g_dbus_proxy_call_sync(powerd_proxy,
+    ret = g_dbus_proxy_call_sync(uscreen_proxy,
             "setUserBrightness",
             g_variant_new("(i)", brightness),
             G_DBUS_CALL_FLAGS_NONE,
             -1, gcancel, &error);
     if (!ret) {
-        g_warning("setUserBrightness via powerd failed: %s", error->message);
+        g_warning("setUserBrightness via unity.screen failed: %s", error->message);
         g_error_free(error);
         return FALSE;
     } else {
@@ -109,9 +128,9 @@ static gboolean setUserBrightness(GDBusProxy* powerd_proxy, GCancellable *gcance
     }
 }
 
-struct _IbBrightnessPowerdControl
+struct _IbBrightnessUScreenControl
 {
-    GDBusProxy *powerd_proxy;
+    GDBusProxy *uscreen_proxy;
     GCancellable *gcancel;
 
     int dim;
@@ -123,13 +142,13 @@ struct _IbBrightnessPowerdControl
     int current;
 };
 
-IbBrightnessPowerdControl*
-ib_brightness_powerd_control_new (GDBusProxy* powerd_proxy, brightness_params_t params)
+IbBrightnessUscreenControl*
+ib_brightness_uscreen_control_new (GDBusProxy* uscreen_proxy, brightness_params_t params)
 {
-    IbBrightnessPowerdControl *control;
+    IbBrightnessUscreenControl *control;
 
-    control = g_new0 (IbBrightnessPowerdControl, 1);
-    control->powerd_proxy = powerd_proxy;
+    control = g_new0 (IbBrightnessUscreenControl, 1);
+    control->uscreen_proxy = uscreen_proxy;
     control->gcancel = g_cancellable_new();
 
     control->dim = params.dim;
@@ -139,21 +158,21 @@ ib_brightness_powerd_control_new (GDBusProxy* powerd_proxy, brightness_params_t
     control->ab_supported = params.ab_supported;
 
     // XXX: set the brightness value is the only way to sync the brightness value with
-    // powerd, and we should set the user prefered / last set brightness value upon startup.
+    // unity.screen, and we should set the user prefered / last set brightness value upon startup.
     // Before we have code to store last set brightness value or other mechanism, we set
     // it to default brightness that powerd proposed.
-    ib_brightness_powerd_control_set_value(control, control->dflt);
+    ib_brightness_uscreen_control_set_value(control, control->dflt);
 
     return control;
 }
 
 void
-ib_brightness_powerd_control_set_value (IbBrightnessPowerdControl* self, gint value)
+ib_brightness_uscreen_control_set_value (IbBrightnessUscreenControl* self, gint value)
 {
     gboolean ret;
 
     value = CLAMP(value, self->min, self->max);
-    ret = setUserBrightness(self->powerd_proxy, self->gcancel, value);
+    ret = setUserBrightness(self->uscreen_proxy, self->gcancel, value);
     if (ret)
     {
         self->current = value;
@@ -161,23 +180,23 @@ ib_brightness_powerd_control_set_value (IbBrightnessPowerdControl* self, gint va
 }
 
 gint
-ib_brightness_powerd_control_get_value (IbBrightnessPowerdControl* self)
+ib_brightness_uscreen_control_get_value (IbBrightnessUscreenControl* self)
 {
     return self->current;
 }
 
 gint
-ib_brightness_powerd_control_get_max_value (IbBrightnessPowerdControl* self)
+ib_brightness_uscreen_control_get_max_value (IbBrightnessUscreenControl* self)
 {
     return self->max;
 }
 
 void
-ib_brightness_powerd_control_free (IbBrightnessPowerdControl *self)
+ib_brightness_uscreen_control_free (IbBrightnessUscreenControl *self)
 {
     g_cancellable_cancel (self->gcancel);
     g_object_unref (self->gcancel);
-    g_object_unref (self->powerd_proxy);
+    g_object_unref (self->uscreen_proxy);
     g_free (self);
 }
 
diff --git a/src/ib-brightness-powerd-control.h b/src/ib-brightness-uscreen-control.h
index 95de1f4..3d026a9 100644
--- a/src/ib-brightness-powerd-control.h
+++ b/src/ib-brightness-uscreen-control.h
@@ -17,8 +17,8 @@
  *     Y.C Cheng <yc.cheng@canonical.com>
  */
 
-#ifndef __IB_BRIGHTNESS_POWERD_CONTROL_H__
-#define __IB_BRIGHTNESS_POWERD_CONTROL_H__
+#ifndef __IB_BRIGHTNESS_USCREEN_CONTROL_H__
+#define __IB_BRIGHTNESS_USCREEN_CONTROL_H__
 
 #include <gio/gio.h>
 
@@ -30,14 +30,14 @@ typedef struct {
     gboolean ab_supported;
 } brightness_params_t;
 
-GDBusProxy* powerd_get_proxy(brightness_params_t *);
+GDBusProxy* uscreen_get_proxy(brightness_params_t *);
 
-typedef struct _IbBrightnessPowerdControl IbBrightnessPowerdControl;
+typedef struct _IbBrightnessUScreenControl IbBrightnessUscreenControl;
 
-IbBrightnessPowerdControl* ib_brightness_powerd_control_new (GDBusProxy* powerd_proxy, brightness_params_t params);
-void                       ib_brightness_powerd_control_set_value (IbBrightnessPowerdControl* self, gint value);
-gint                       ib_brightness_powerd_control_get_value (IbBrightnessPowerdControl* self);
-gint                       ib_brightness_powerd_control_get_max_value (IbBrightnessPowerdControl* self);
-void                       ib_brightness_powerd_control_free (IbBrightnessPowerdControl *self);
+IbBrightnessUscreenControl* ib_brightness_uscreen_control_new (GDBusProxy* uscreen_proxy, brightness_params_t params);
+void                       ib_brightness_uscreen_control_set_value (IbBrightnessUscreenControl* self, gint value);
+gint                       ib_brightness_uscreen_control_get_value (IbBrightnessUscreenControl* self);
+gint                       ib_brightness_uscreen_control_get_max_value (IbBrightnessUscreenControl* self);
+void                       ib_brightness_uscreen_control_free (IbBrightnessUscreenControl *self);
 
 #endif
diff --git a/src/service.c b/src/service.c
index 46ad802..6438c9a 100644
--- a/src/service.c
+++ b/src/service.c
@@ -27,7 +27,7 @@
 #include "device.h"
 #include "device-provider.h"
 #include "ib-brightness-control.h"
-#include "ib-brightness-powerd-control.h"
+#include "ib-brightness-uscreen-control.h"
 #include "service.h"
 
 #define BUS_NAME "com.canonical.indicator.power"
@@ -105,7 +105,7 @@ struct _IndicatorPowerServicePrivate
   GSettings * settings;
 
   IbBrightnessControl * brightness_control;
-  IbBrightnessPowerdControl * brightness_powerd_control;
+  IbBrightnessUscreenControl * brightness_uscreen_control;
 
   guint own_id;
   guint actions_export_id;
@@ -467,9 +467,9 @@ get_brightness_range (IndicatorPowerService * self, gint * low, gint * high)
     {
       max = ib_brightness_control_get_max_value (self->priv->brightness_control);
     }
-  else if (p->brightness_powerd_control)
+  else if (p->brightness_uscreen_control)
     {
-      max = ib_brightness_powerd_control_get_max_value (self->priv->brightness_powerd_control);
+      max = ib_brightness_uscreen_control_get_max_value (self->priv->brightness_uscreen_control);
     }
   *low  = max * 0.05; /* 5% minimum -- don't let the screen go completely dark */
   *high = max;
@@ -500,9 +500,9 @@ action_state_for_brightness (IndicatorPowerService * self)
     {
       brightness = ib_brightness_control_get_value (p->brightness_control);
     }
-  else if (p->brightness_powerd_control)
+  else if (p->brightness_uscreen_control)
     {
-      brightness = ib_brightness_powerd_control_get_value (p->brightness_powerd_control);
+      brightness = ib_brightness_uscreen_control_get_value (p->brightness_uscreen_control);
     }
   return g_variant_new_double (brightness_to_percentage (self, brightness));
 }
@@ -527,9 +527,9 @@ on_brightness_change_requested (GSimpleAction * action      G_GNUC_UNUSED,
     {
       ib_brightness_control_set_value (self->priv->brightness_control, brightness);
     }
-  else if (self->priv->brightness_powerd_control)
+  else if (self->priv->brightness_uscreen_control)
     {
-      ib_brightness_powerd_control_set_value (self->priv->brightness_powerd_control, brightness);
+      ib_brightness_uscreen_control_set_value (self->priv->brightness_uscreen_control, brightness);
     }
 
   update_brightness_action_state (self);
@@ -1024,7 +1024,7 @@ my_dispose (GObject * o)
 
   // g_clear_pointer has NULL check inside.
   g_clear_pointer (&p->brightness_control, ib_brightness_control_free);
-  g_clear_pointer (&p->brightness_powerd_control, ib_brightness_powerd_control_free);
+  g_clear_pointer (&p->brightness_uscreen_control, ib_brightness_uscreen_control_free);
 
   indicator_power_service_set_device_provider (self, NULL);
 
@@ -1038,8 +1038,8 @@ my_dispose (GObject * o)
 static void
 indicator_power_service_init (IndicatorPowerService * self)
 {
-  GDBusProxy *powerd_proxy;
-  brightness_params_t powerd_brightness_params;
+  GDBusProxy *uscreen_proxy;
+  brightness_params_t brightness_params;
   priv_t * p = G_TYPE_INSTANCE_GET_PRIVATE (self,
                                             INDICATOR_TYPE_POWER_SERVICE,
                                             IndicatorPowerServicePrivate);
@@ -1049,10 +1049,10 @@ indicator_power_service_init (IndicatorPowerService * self)
 
   p->settings = g_settings_new ("com.canonical.indicator.power");
 
-  powerd_proxy = powerd_get_proxy(&powerd_brightness_params);
-  if (powerd_proxy != NULL)
+  uscreen_proxy = uscreen_get_proxy(&brightness_params);
+  if (uscreen_proxy != NULL)
     {
-      p->brightness_powerd_control = ib_brightness_powerd_control_new(powerd_proxy, powerd_brightness_params);
+      p->brightness_uscreen_control = ib_brightness_uscreen_control_new(uscreen_proxy, brightness_params);
     }
   else
     {