From 845067b3f37ff48eab3b612d52518e529559b738 Mon Sep 17 00:00:00 2001
From: Charles Kerr <charles.kerr@canonical.com>
Date: Tue, 31 Mar 2015 13:54:26 -0500
Subject: simplify the awake display timeout code

---
 include/notifications/awake.h |  2 +-
 src/awake.cpp                 | 23 ++++++++++++++++-------
 src/snap.cpp                  |  7 ++-----
 3 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/include/notifications/awake.h b/include/notifications/awake.h
index ad04cca..bc38817 100644
--- a/include/notifications/awake.h
+++ b/include/notifications/awake.h
@@ -36,7 +36,7 @@ namespace notifications {
 class Awake
 {
 public:
-    explicit Awake(const std::string& app_name, unsigned int display_on_seconds);
+    explicit Awake(const std::string& app_name);
     ~Awake();
 
 private:
diff --git a/src/awake.cpp b/src/awake.cpp
index a37d185..e1bec6c 100644
--- a/src/awake.cpp
+++ b/src/awake.cpp
@@ -36,10 +36,9 @@ class Awake::Impl
 {
 public:
 
-    Impl(const std::string& app_name, unsigned int display_on_seconds):
+    Impl(const std::string& app_name):
         m_app_name(app_name),
-        m_cancellable(g_cancellable_new()),
-        m_display_on_seconds(display_on_seconds)
+        m_cancellable(g_cancellable_new())
     {
         g_bus_get(G_BUS_TYPE_SYSTEM, m_cancellable, on_system_bus_ready, this);
     }
@@ -251,9 +250,19 @@ private:
     GCancellable * m_cancellable = nullptr;
     GDBusConnection * m_system_bus = nullptr;
     char * m_awake_cookie = nullptr;
+
+    /**
+     * As described by bug #1434637, alarms should have the display turn on,
+     * dim, and turn off "just like it would if you'd woken it up yourself".
+     * USC may be adding an intent-based bus API to handle this use case,
+     * e.g. turnDisplayOnTemporarily(intent), but there's no timeframe for it.
+     *
+     * Until that's avaialble, we can get close to Design's specs by
+     * requesting a display-on cookie and then releasing the cookie
+     * a moment later. */
+    const guint m_display_on_seconds = 1;
+    guint m_display_on_timer = 0;
     int32_t m_display_on_cookie = NO_DISPLAY_ON_COOKIE;
-    const guint m_display_on_seconds;
-    guint m_display_on_timer;
 
     static constexpr int32_t NO_DISPLAY_ON_COOKIE { std::numeric_limits<int32_t>::min() };
 };
@@ -262,8 +271,8 @@ private:
 ****
 ***/
 
-Awake::Awake(const std::string& app_name, unsigned int display_on_seconds):
-    impl(new Impl (app_name, display_on_seconds))
+Awake::Awake(const std::string& app_name):
+    impl(new Impl(app_name))
 {
 }
 
diff --git a/src/snap.cpp b/src/snap.cpp
index 4329eca..e655d2d 100644
--- a/src/snap.cpp
+++ b/src/snap.cpp
@@ -88,11 +88,8 @@ public:
            intervention and shouldn't loop the sound. */
         const bool interactive = appointment.is_ubuntu_alarm() && m_engine->supports_actions();
 
-        // keep the screen on for the first part of the alarm;
-        // keep the system awake for the duration of the alarm
-        constexpr unsigned int display_on_seconds = 60;
-        auto awake = std::make_shared<uin::Awake>(m_engine->app_name(),
-                                                  display_on_seconds);
+        // force the system to stay awake
+        auto awake = std::make_shared<uin::Awake>(m_engine->app_name());
 
         // calendar events are muted in silent mode; alarm clocks never are
         std::shared_ptr<uin::Sound> sound;
-- 
cgit v1.2.3