Merge lp:~charlesk/indicator-power/lp-1296431-low-power-snap-decisions into lp:indicator-power/14.10

[Charles Kerr]

When the phone's battery goes down past a certain level, pop up a snap decision to warn the user.

== Description of the Change

When the battery charge level goes down past the predefined thresholds,

show this on the phone: https://launchpadlibrarian.net/183295784/battery%20low%20notification.jpg

and this on the desktop: https://wiki.ubuntu.com/Power?action=AttachFile&do=get&target=battery-low-pc.png

NB: the phone warnings wireframe at https://wiki.ubuntu.com/Power#Warnings is not up-to-date. Phone warnings should look like the wireframe above wrt being a snap decision with options to dismiss or to launch battery settings.

== Checklist

> Are there any related MPs required for this MP to build/function as expected? Please list.

No

> Is your branch in sync with latest trunk? (e.g. bzr pull lp:trunk -> no changes)

Yes

> Did the code build without warnings?

Yes

> Did the tests run successfully?

Yes

> Did you perform an exploratory manual test run of your code change and any related functionality?

Yes

> If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?

N/A

> Did your component test plan pass? If on a device, what image number?

mako + rtm r29

> Did you provide a link to this page https://wiki.ubuntu.com/Process/Merges/Checklists/indicator-power

Yes

> Please list which manual tests are germane for the reviewer in this MR.

New test: indicator-power/low-power-notifications

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:258
http://jenkins.qa.ubuntu.com/job/indicator-power-ci/127/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-amd64-ci/28/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-armhf-ci/28
        deb: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-armhf-ci/28/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/indicator-power-ci/127/rebuild

review: Needs Fixing (continuous-integration)

[lp:~charlesk/indicator-power/lp-1296431-low-power-snap-decisions]

259. By Charles Kerr on 2014-09-08

in notifier.c, reverse the order in which we add the actions so they'll look right on the phone

260. By Charles Kerr on 2014-09-08

in notifier.c, don't call notify_get_server_caps() if notify_init() failed.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:258
http://jenkins.qa.ubuntu.com/job/indicator-power-ci/130/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-amd64-ci/31
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-armhf-ci/31
        deb: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-armhf-ci/31/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/indicator-power-ci/130/rebuild

review: Approve (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:260
http://jenkins.qa.ubuntu.com/job/indicator-power-ci/131/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-amd64-ci/32
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-armhf-ci/32
        deb: http://jenkins.qa.ubuntu.com/job/indicator-power-utopic-armhf-ci/32/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/indicator-power-ci/131/rebuild

review: Approve (continuous-integration)

[Ted Gould (ted)]

Bad design, but eh, I guess we can't fix that here.

We're going to need to track the notification and close it when the next one comes in. Probably that's the standard case, people aren't using their phone and just getting notifications that aren't useful, we don't want them to stack up.

review: Needs Fixing

[lp:~charlesk/indicator-power/lp-1296431-low-power-snap-decisions]

261. By Charles Kerr on 2014-09-08

in notifier's snap decisions, distinguish in the title between low battery and critical battery

[Ted Gould (ted)]

Cool, explained to me on IRC.

review: Approve

[lp:~charlesk/indicator-power/lp-1296431-low-power-snap-decisions]

262. By Charles Kerr on 2014-09-08

sync with lp-1330037-add-upower-099-support to resolve merge conflicts

=== modified file 'src/notifier.c'
--- src/notifier.c	2014-07-25 05:34:59 +0000
+++ src/notifier.c	2014-09-08 21:23:16 +0000
@@ -21,11 +21,13 @@
 #include "dbus-shared.h"
 #include "notifier.h"
 
+#include <url-dispatcher.h>
+
 #include <libnotify/notify.h>
 
 #include <glib/gi18n.h>
 
-#define HINT_INTERACTIVE "x-canonical-switch-to-application"
+#include <stdint.h> /* UINT32_MAX */
 
 typedef enum
 {
@@ -53,6 +55,8 @@
 
 static int instance_count = 0;
 
+static gboolean actions_supported = FALSE;
+
 /**
 ***
 **/
@@ -99,7 +103,7 @@
     }
 }
 
-PowerLevel
+static PowerLevel
 get_battery_power_level (IndicatorPowerDevice * battery)
 {
   static const double percent_critical = 2.0;
@@ -163,22 +167,62 @@
 }
 
 static void
+on_battery_settings_clicked(NotifyNotification * nn        G_GNUC_UNUSED,
+                            char               * action    G_GNUC_UNUSED,
+                            gpointer             user_data G_GNUC_UNUSED)
+{
+  url_dispatch_send("settings:///system/battery", NULL, NULL);
+}
+
+static void
+on_dismiss_clicked(NotifyNotification * nn        G_GNUC_UNUSED,
+                   char               * action    G_GNUC_UNUSED,
+                   gpointer             user_data G_GNUC_UNUSED)
+{
+  /* no-op; libnotify warns if we have a NULL action callback */
+}
+
+static void
 notification_show(IndicatorPowerNotifier * self)
 {
   priv_t * const p = get_priv(self);
   gdouble pct;
+  const char * title;
   char * body;
+  GStrv icon_names;
+  const char * icon_name;
   NotifyNotification * nn;
   GError * error;
+  const PowerLevel power_level = get_battery_power_level(p->battery);
 
   notification_clear(self);
 
+  g_return_if_fail(power_level != POWER_LEVEL_OK);
+
   /* create the notification */
+  title = power_level == POWER_LEVEL_LOW
+        ? _("Battery Low")
+        : _("Battery Critical");
   pct = indicator_power_device_get_percentage(p->battery);
   body = g_strdup_printf(_("%.0f%% charge remaining"), pct);
-  nn = notify_notification_new(_("Battery Low"), body, NULL);
+  icon_names = indicator_power_device_get_icon_names(p->battery);
+  if (icon_names && *icon_names)
+    icon_name = icon_names[0];
+  else
+    icon_name = NULL;
+  nn = notify_notification_new(title, body, icon_name);
+  g_strfreev (icon_names);
   g_free (body);
-  /*notify_notification_set_hint(nn, HINT_INTERACTIVE, g_variant_new_boolean(TRUE));*/
+
+  if (actions_supported)
+    {
+      notify_notification_set_hint(nn, "x-canonical-snap-decisions", g_variant_new_boolean(TRUE));
+      notify_notification_set_hint(nn, "x-canonical-non-shaped-icon", g_variant_new_boolean(TRUE));
+      notify_notification_set_hint(nn, "x-canonical-snap-decisions-timeout", g_variant_new_int32(INT32_MAX));
+      notify_notification_set_timeout(nn, NOTIFY_EXPIRES_NEVER);
+      notify_notification_add_action(nn, "dismiss", _("OK"), on_dismiss_clicked, NULL, NULL);
+      notify_notification_add_action(nn, "settings", _("Battery settings"), on_battery_settings_clicked, NULL, NULL);
+    }
 
   /* if we can show it, keep it */
   error = NULL;
@@ -322,10 +366,24 @@
 
   if (!instance_count++)
     {
+      actions_supported = FALSE;
+
       if (!notify_init("indicator-power-service"))
         {
           g_critical("Unable to initialize libnotify! Notifications might not be shown.");
         }
+      else
+        {
+          GList * caps;
+          GList * l;
+
+          /* see if actions are supported */
+          caps = notify_get_server_caps();
+          for (l=caps; l!=NULL && !actions_supported; l=l->next)
+            if (!g_strcmp0(l->data, "actions"))
+              actions_supported = TRUE;
+          g_list_free_full(caps, g_free);
+        }
     }
 }
 

=== modified file 'tests/indicator-power-service-cmdline-battery.cc'
--- tests/indicator-power-service-cmdline-battery.cc	2014-07-22 14:58:29 +0000
+++ tests/indicator-power-service-cmdline-battery.cc	2014-09-08 21:23:16 +0000
@@ -83,10 +83,13 @@
 int
 main (int argc G_GNUC_UNUSED, char ** argv G_GNUC_UNUSED)
 {
-  g_message ("This app is basically the same as indicator-power-service but,\n"
-             "instead of the system's real devices, sees a single fake battery\n"
-             "which can be manipulated by typing commands:\n"
-             "'charging', 'discharging', a charge percentage, or ctrl-c.");
+  g_print("This test app has the same code as indicator-power-service\n"
+          "except instead of listening to UPower, it has a fake battery\n"
+          "which you can edit with keyboard inputs. Supported commands:\n"
+          "1. A number in [0..100] to set battery level\n"
+          "2. 'charging'\n"
+          "3. 'discharging'\n"
+          "4. ctrl-c to exit\n");
 
   IndicatorPowerDeviceProvider * device_provider;
   IndicatorPowerService * service;