Merge lp:~3v1n0/bamf/running-applications-changed-signal-fix into lp:bamf/0.4

[Marco Trevisan (Treviño)]

BamfMatcher must emit the "running-applications-changed" signal for closed applications.

BamfMatcher must emit the "running-applications-changed" signal for closed applications.

[Michal Hruby (mhr3)]

+1, the signals are now properly emitted. But we should have a test for this to prevent regressions.

[Gord Allott (gordallott)]

review: Approve

[Unity Merger (unity-merger)]

No commit message specified.

=== modified file 'src/bamf-matcher.c'
--- src/bamf-matcher.c	2012-04-20 17:36:00 +0000
+++ src/bamf-matcher.c	2012-04-27 11:57:18 +0000
@@ -210,6 +210,7 @@
 static void bamf_matcher_prepare_path_change (BamfMatcher *self, const gchar *desktop_file, ViewChangeType change_type)
 {
   BamfMatcherPrivate *priv;
+  BamfApplication *app;
 
   if (desktop_file == NULL) return;
 
@@ -219,7 +220,9 @@
 
   /* the app was already running (ADDED) / had more instances which are still
    * there (REMOVED) */
-  if (bamf_matcher_get_application_by_desktop_file (self, desktop_file))
+  app = bamf_matcher_get_application_by_desktop_file (self, desktop_file);
+
+  if (BAMF_IS_APPLICATION (app) && bamf_view_is_running (BAMF_VIEW (app)))
     {
       return;
     }
@@ -1528,7 +1531,7 @@
 is_web_app_window (BamfMatcher *self, BamfLegacyWindow *window)
 {
   const char *window_class = bamf_legacy_window_get_class_name (window);
-  const char *instance_name = bamf_legacy_window_get_class_instance_name(window);
+  const char *instance_name = bamf_legacy_window_get_class_instance_name (window);
 
   // Chrome/Chromium uses url wm_class strings to represent its web apps.
   // These apps will still have the same parent pid and hints as the main chrome
@@ -1639,7 +1642,7 @@
     {
       desktop_class = g_hash_table_lookup (priv->desktop_class_table, desktop_file);
 
-      if (!known_desktop_class || g_strcmp0 (desktop_class, desktop_file) == 0)
+      if (!known_desktop_class || g_strcmp0 (desktop_class, instance_name) == 0)
         {
           desktop_files = g_list_prepend (desktop_files, desktop_file);
         }
@@ -1730,7 +1733,7 @@
                             }
                         }
                     }
-                  
+
                   desktop_files = g_list_insert_before (desktop_files, last, desktop_file);
                 }
               else