Merge lp:~azzar1/bamf/manually-complete-sn into lp:bamf

[Andrea Azzarone]

Daemon: manually set to complete the startup notification on view state change

Specifically when the view is running, activated or marked as urgent.

[Marco Trevisan (Treviño) (3v1n0)]

Looks good.

review: Approve

=== modified file 'src/bamf-control.c'
--- src/bamf-control.c	2016-03-21 15:17:45 +0000
+++ src/bamf-control.c	2016-06-13 22:30:23 +0000
@@ -54,7 +54,7 @@
   g_variant_get_child (parameters, 0, "^&ay", &desktop_file);
   g_variant_get_child (parameters, 2, "x", &pid);
 
-  bamf_matcher_set_starting_desktop_file (matcher, desktop_file, TRUE);
+  bamf_matcher_set_starting_desktop_file (matcher, desktop_file, NULL, TRUE);
   bamf_matcher_register_desktop_file_for_pid (matcher, desktop_file, pid);
 }
 
 
=== modified file 'src/bamf-legacy-screen.c'
--- src/bamf-legacy-screen.c	2016-03-21 15:21:45 +0000
+++ src/bamf-legacy-screen.c	2016-06-13 22:30:23 +0000
@@ -258,7 +258,7 @@
       {
         SnStartupSequence *sequence = sn_monitor_event_get_startup_sequence (event);
         const gchar *app_id = sn_startup_sequence_get_application_id (sequence);
-        g_signal_emit (self, legacy_screen_signals[WINDOW_OPENING], 0, app_id);
+        g_signal_emit (self, legacy_screen_signals[WINDOW_OPENING], 0, app_id, sequence);
       }
       break;
     case SN_MONITOR_EVENT_COMPLETED:
@@ -477,8 +477,8 @@
                   G_SIGNAL_RUN_FIRST,
                   G_STRUCT_OFFSET (BamfLegacyScreenClass, window_opening),
                   NULL, NULL, NULL,
-                  G_TYPE_NONE, 1,
-                  G_TYPE_STRING);
+                  G_TYPE_NONE, 2,
+                  G_TYPE_STRING, G_TYPE_POINTER);
 
   legacy_screen_signals [WINDOW_OPENING_COMPLETED] =
     g_signal_new (BAMF_LEGACY_SCREEN_SIGNAL_WINDOW_OPENING_COMPLETED,
 
=== modified file 'src/bamf-matcher.c'
--- src/bamf-matcher.c	2016-06-01 11:57:58 +0000
+++ src/bamf-matcher.c	2016-06-13 22:30:23 +0000
@@ -2250,15 +2250,15 @@
 }
 
 static void
-handle_window_opening (BamfLegacyScreen *screen, const gchar *desktop_id, BamfMatcher *self)
+handle_window_opening (BamfLegacyScreen *screen, const gchar *desktop_id, SnStartupSequence *startup_sequence, BamfMatcher *self)
 {
-  bamf_matcher_set_starting_desktop_file (self, desktop_id, TRUE);
+  bamf_matcher_set_starting_desktop_file (self, desktop_id, startup_sequence, TRUE);
 }
 
 static void
 handle_window_opening_finished (BamfLegacyScreen *screen, const gchar *desktop_id, BamfMatcher *self)
 {
-  bamf_matcher_set_starting_desktop_file (self, desktop_id, FALSE);
+  bamf_matcher_set_starting_desktop_file (self, desktop_id, NULL, FALSE);
 }
 
 static void
@@ -2499,6 +2499,7 @@
 void
 bamf_matcher_set_starting_desktop_file (BamfMatcher *self,
                                         const char *desktop_file,
+                                        SnStartupSequence *startup_sequence,
                                         gboolean starting)
 {
   BamfApplication *app;
@@ -2523,7 +2524,7 @@
 
   if (BAMF_IS_APPLICATION (app))
     {
-      bamf_view_set_starting (BAMF_VIEW (app), starting);
+      bamf_view_set_starting (BAMF_VIEW (app), startup_sequence, starting);
     }
 }
 
 
=== modified file 'src/bamf-matcher.h'
--- src/bamf-matcher.h	2016-03-21 15:16:22 +0000
+++ src/bamf-matcher.h	2016-06-13 22:30:23 +0000
@@ -76,6 +76,7 @@
 
 void          bamf_matcher_set_starting_desktop_file     (BamfMatcher * self,
                                                           const char *desktop_file,
+                                                          SnStartupSequence *sequence,
                                                           gboolean starting);
 
 const char  * bamf_matcher_get_desktop_file_class        (BamfMatcher * self,
 
=== modified file 'src/bamf-view.c'
--- src/bamf-view.c	2016-03-29 14:52:33 +0000
+++ src/bamf-view.c	2016-06-13 22:30:23 +0000
@@ -78,6 +78,8 @@
   gboolean closed;
   guint starting_timeout;
 
+  SnStartupSequence *startup_sequence;
+
   /* FIXME: remove this as soon as we move to properties on library as well */
   guint active_changed_idle;
 };
@@ -115,6 +117,9 @@
       guint idle = g_idle_add_full (G_PRIORITY_DEFAULT, on_active_changed_idle, view, NULL);
       view->priv->active_changed_idle = idle;
     }
+
+  if (active)
+    bamf_view_set_starting (view, NULL, FALSE);
 }
 
 static void
@@ -152,7 +157,7 @@
 {
   BamfView *view = data;
 
-  bamf_view_set_starting (view, FALSE);
+  bamf_view_set_starting (view, NULL, FALSE);
   view->priv->starting_timeout = 0;
 
   return FALSE;
@@ -197,7 +202,7 @@
     g_signal_emit_by_name (view, "running-changed", running);
 
   if (running)
-    bamf_view_set_starting (view, FALSE);
+    bamf_view_set_starting (view, NULL, FALSE);
 }
 
 static void
@@ -213,6 +218,9 @@
 
   if (emit)
     g_signal_emit_by_name (view, "urgent-changed", urgent);
+
+  if (urgent)
+    bamf_view_set_starting (view, NULL, FALSE);
 }
 
 void
@@ -480,8 +488,33 @@
 
 void
 bamf_view_set_starting (BamfView *view,
+                        SnStartupSequence *startup_sequence,
                         gboolean starting)
 {
+  if (!bamf_view_is_starting (view) && starting)
+    {
+      if (view->priv->startup_sequence)
+      {
+        sn_startup_sequence_unref (view->priv->startup_sequence);
+        view->priv->startup_sequence = NULL;
+      }
+
+      if (startup_sequence)
+        {
+          view->priv->startup_sequence = startup_sequence;
+          sn_startup_sequence_ref (view->priv->startup_sequence);
+        }
+    }
+  else if (!starting)
+    {
+      if (view->priv->startup_sequence)
+        {
+          sn_startup_sequence_complete (view->priv->startup_sequence);
+          sn_startup_sequence_unref (view->priv->startup_sequence);
+          view->priv->startup_sequence = NULL;
+        }
+    }
+
   BAMF_VIEW_SET_BOOL_PROPERTY (view, starting);
 }
 
@@ -581,7 +614,7 @@
   bamf_view_set_name (view, cache->name);
   bamf_view_set_icon (view, cache->icon);
   bamf_view_set_active (view, cache->active);
-  bamf_view_set_starting (view, cache->starting);
+  bamf_view_set_starting (view, NULL, cache->starting);
   bamf_view_set_running (view, cache->running);
   bamf_view_set_user_visible (view, cache->user_visible);
   bamf_view_set_urgent (view, cache->urgent);
@@ -863,6 +896,12 @@
       priv->active_changed_idle = 0;
     }
 
+  if (priv->startup_sequence)
+    {
+      sn_startup_sequence_unref (priv->startup_sequence);
+      priv->startup_sequence = NULL;
+    }
+
   bamf_view_cached_properties_clear (view);
   g_dbus_object_skeleton_flush (G_DBUS_OBJECT_SKELETON (view));
 
@@ -931,6 +970,8 @@
   self->priv->dbus_iface = _bamf_dbus_item_view_skeleton_new ();
   self->priv->props = g_new0 (BamfViewPropCache, 1);
 
+  self->priv->startup_sequence = NULL;
+
   /* We need to connect to the object own signals to redirect them to the dbus
    * interface                                                                */
   g_signal_connect (self, "active-changed", G_CALLBACK (on_view_active_changed), NULL);
 
=== modified file 'src/bamf-view.h'
--- src/bamf-view.h	2016-03-29 14:32:40 +0000
+++ src/bamf-view.h	2016-06-13 22:30:23 +0000
@@ -25,6 +25,10 @@
 #include <glib-object.h>
 #include <libbamf-private/bamf-private.h>
 
+#define SN_API_NOT_YET_FROZEN
+#include <libsn/sn.h>
+#undef SN_API_NOT_YET_FROZEN
+
 #define BAMF_TYPE_VIEW                  (bamf_view_get_type ())
 #define BAMF_VIEW(obj)                  (G_TYPE_CHECK_INSTANCE_CAST ((obj), BAMF_TYPE_VIEW, BamfView))
 #define BAMF_IS_VIEW(obj)               (G_TYPE_CHECK_INSTANCE_TYPE ((obj), BAMF_TYPE_VIEW))
@@ -89,7 +93,7 @@
 void          bamf_view_set_active         (BamfView *view, gboolean active);
 
 gboolean      bamf_view_is_starting        (BamfView *view);
-void          bamf_view_set_starting       (BamfView *view, gboolean starting);
+void          bamf_view_set_starting       (BamfView *view, SnStartupSequence *startup_sequence, gboolean starting);
 
 gboolean      bamf_view_is_running         (BamfView *view);
 void          bamf_view_set_running        (BamfView *view, gboolean running);