[gnome-shell] Bug 592402 - Better handling of Nautilus window

[Colin Walters <walters src gnome org>]

commit 62b2d69c2b6991cc722f56056c2236e58d1cf718
Author: Colin Walters <walters verbum org>
Date:   Thu Aug 20 15:45:12 2009 -0400

    Bug 592402 - Better handling of Nautilus window
    
    Separate the application monitor logic for "tracking" and "usage tracking".
    The first means we associate an application with a window.  The second
    means we count focus time inside that window, and consider the window
    interesting from a user point of view.
    
    (Really, should probably split ShellAppMonitor into two classes along
     this line, with the second consuming the first).
    
    For the purposes of counting running applications and returning
    the list of open windows for an application, skip not-usage-tracked
    windows.
    
    Together this allows us to associate the Nautilus desktop window
    with the nautilus.desktop, but not show "File Manager" open all
    of the time.

 src/shell-app-monitor.c |  107 ++++++++++++++++++++++++++++++++++++++++-------
 1 files changed, 92 insertions(+), 15 deletions(-)
---
diff --git a/src/shell-app-monitor.c b/src/shell-app-monitor.c
index 3ee4acd..caefcd2 100644
--- a/src/shell-app-monitor.c
+++ b/src/shell-app-monitor.c
@@ -323,10 +323,67 @@ get_cleaned_wmclass_for_window (MetaWindow  *window)
   return wmclass;
 }
 
+/**
+ * window_is_tracked:
+ *
+ * Returns: %TRUE iff we want to scan this window for application association
+ */
 static gboolean
 window_is_tracked (MetaWindow *window)
 {
-  return !meta_window_is_override_redirect (window);
+  if (meta_window_is_override_redirect (window))
+    return FALSE;
+
+  return TRUE;
+}
+
+/**
+ * window_is_usage_tracked:
+ *
+ * Determine if it makes sense to track the given window for application
+ * usage.  An example of a window we don't want to track is the root
+ * desktop window.  We skip all override-redirect types, and also
+ * exclude other window types like tooltip explicitly, though generally
+ * most of these should be override-redirect.
+ *
+ * The usage data is also currently used to return the list of user-interesting
+ * windows associated with an application.
+ *
+ * Returns: %TRUE iff we want to record focus time spent in this window
+ */
+static gboolean
+window_is_usage_tracked (MetaWindow *window)
+{
+  if (!window_is_tracked (window))
+    return FALSE;
+
+  switch (meta_window_get_window_type (window))
+    {
+      /* Definitely ignore these. */
+      case META_WINDOW_DESKTOP:
+      case META_WINDOW_DOCK:
+      case META_WINDOW_SPLASHSCREEN:
+      /* Should have already been handled by override_redirect above,
+       * but explicitly list here so we get the "unhandled enum"
+       * warning if in the future anything is added.*/
+      case META_WINDOW_DROPDOWN_MENU:
+      case META_WINDOW_POPUP_MENU:
+      case META_WINDOW_TOOLTIP:
+      case META_WINDOW_NOTIFICATION:
+      case META_WINDOW_COMBO:
+      case META_WINDOW_DND:
+      case META_WINDOW_OVERRIDE_OTHER:
+        return FALSE;
+      case META_WINDOW_NORMAL:
+      case META_WINDOW_DIALOG:
+      case META_WINDOW_MODAL_DIALOG:
+      case META_WINDOW_MENU:
+      case META_WINDOW_TOOLBAR:
+      case META_WINDOW_UTILITY:
+        break;
+    }
+
+  return TRUE;
 }
 
 static ShellAppInfo *
@@ -491,7 +548,7 @@ get_active_window (ShellAppMonitor *monitor)
   display = meta_screen_get_display (screen);
   window = meta_display_get_focus_window (display);
 
-  if (window != NULL && window_is_tracked (window))
+  if (window != NULL && window_is_usage_tracked (window))
     return window;
   return NULL;
 }
@@ -648,8 +705,17 @@ track_window (ShellAppMonitor *self,
   if (!app)
     return;
 
+  /* At this point we've stored the association from window -> application */
   g_hash_table_insert (self->window_to_app, window, app);
 
+  /* However, we don't want to record usage for all kinds of windows;
+   * the desktop window is a prime example.  If a window isn't usage
+   * tracked it doesn't count for the purposes of an application
+   * running.
+   */
+  if (!window_is_usage_tracked (window))
+    return;
+
   usage = get_app_usage_from_window (self, window);
   usage->transient = shell_app_info_is_transient (app);
 
@@ -684,8 +750,6 @@ shell_app_monitor_on_window_removed (MetaWorkspace   *workspace,
 {
   ShellAppMonitor *self = SHELL_APP_MONITOR (user_data);
   ShellAppInfo *app;
-  AppUsage *usage;
-  const char *context;
 
   app = g_hash_table_lookup (self->window_to_app, window);
   if (!app)
@@ -693,23 +757,30 @@ shell_app_monitor_on_window_removed (MetaWorkspace   *workspace,
 
   shell_app_info_ref (app);
 
-  context = get_window_context (window);
-  usage = get_app_usage_from_window (self, window);
-
   if (window == self->watched_window)
     self->watched_window = NULL;
 
-  usage->window_count--;
-  /* Remove before emitting */
-  g_hash_table_remove (self->window_to_app, window);
+  if (window_is_usage_tracked (window))
+    {
+      AppUsage *usage;
+      const char *context;
+
+      context = get_window_context (window);
+      usage = get_app_usage_from_window (self, window);
+      usage->window_count--;
 
-  g_signal_emit (self, signals[WINDOW_REMOVED], 0, app, window);
+      g_hash_table_remove (self->window_to_app, window);
 
-  if (usage->window_count == 0)
-    {
-      g_signal_emit (self, signals[APP_REMOVED], 0, app);
-      reset_usage (self, context, shell_app_info_get_id (app), usage);
+      g_signal_emit (self, signals[WINDOW_REMOVED], 0, app, window);
+
+      if (usage->window_count == 0)
+        {
+          g_signal_emit (self, signals[APP_REMOVED], 0, app);
+          reset_usage (self, context, shell_app_info_get_id (app), usage);
+        }
     }
+  else
+    g_hash_table_remove (self->window_to_app, window);
 
   shell_app_info_unref (app);
 }
@@ -799,6 +870,9 @@ shell_app_monitor_get_windows_for_app (ShellAppMonitor *self,
       ShellAppInfo *app = value;
       const char *id = shell_app_info_get_id (app);
 
+      if (!window_is_usage_tracked (window))
+        continue;
+
       if (strcmp (id, appid) != 0)
         continue;
 
@@ -1047,6 +1121,9 @@ shell_app_monitor_get_running_apps (ShellAppMonitor *monitor,
       if (strcmp (get_window_context (window), context) != 0)
         continue;
 
+      if (!window_is_usage_tracked (window))
+        continue;
+
       id = shell_app_info_get_id (app);
 
       if (g_hash_table_lookup (unique_apps, id))