[gnome-shell] shell-app: Track all application windows



commit d21aa0d85fc3251176a7d7f31a12a4b44fcfa58e
Author: Florian Müllner <fmuellner gnome org>
Date:   Wed Dec 4 17:26:01 2013 +0100

    shell-app: Track all application windows
    
    Filtering out "non-interesting" windows beforehand as we currently do
    means that we may get properties that should be based on all windows,
    like the last time the application was used, wrong.
    Just track all windows and filter out non-interesting windows manually
    in the one place we actually care about the difference.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=719824

 js/ui/appDisplay.js        |    4 +++-
 src/shell-app.c            |    8 ++++----
 src/shell-window-tracker.c |   10 ++--------
 3 files changed, 9 insertions(+), 13 deletions(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index ff9c14b..357fcfb 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -1514,7 +1514,9 @@ const AppIconMenu = new Lang.Class({
     _redisplay: function() {
         this.removeAll();
 
-        let windows = this._source.app.get_windows();
+        let windows = this._source.app.get_windows().filter(function(w) {
+            return Shell.WindowTracker.is_window_interesting(w);
+        });
 
         // Display the app windows menu items and the separator between windows
         // of the current desktop and other windows.
diff --git a/src/shell-app.c b/src/shell-app.c
index 63f9bc7..e2745d3 100644
--- a/src/shell-app.c
+++ b/src/shell-app.c
@@ -730,10 +730,10 @@ shell_app_compare_windows (gconstpointer   a,
  * shell_app_get_windows:
  * @app:
  *
- * Get the toplevel, interesting windows which are associated with this
- * application.  The returned list will be sorted first by whether
- * they're on the active workspace, then by whether they're visible,
- * and finally by the time the user last interacted with them.
+ * Get the windows which are associated with this application. The
+ * returned list will be sorted first by whether they're on the
+ * active workspace, then by whether they're visible, and finally
+ * by the time the user last interacted with them.
  *
  * Returns: (transfer none) (element-type MetaWindow): List of windows
  */
diff --git a/src/shell-window-tracker.c b/src/shell-window-tracker.c
index e1d563f..4bccf6e 100644
--- a/src/shell-window-tracker.c
+++ b/src/shell-window-tracker.c
@@ -489,9 +489,6 @@ track_window (ShellWindowTracker *self,
 {
   ShellApp *app;
 
-  if (!shell_window_tracker_is_window_interesting (window))
-    return;
-
   app = get_app_for_window (self, window);
   if (!app)
     return;
@@ -530,11 +527,8 @@ disassociate_window (ShellWindowTracker   *self,
 
   g_hash_table_remove (self->window_to_app, window);
 
-  if (shell_window_tracker_is_window_interesting (window))
-    {
-      _shell_app_remove_window (app, window);
-      g_signal_handlers_disconnect_by_func (window, G_CALLBACK(on_wm_class_changed), self);
-    }
+  _shell_app_remove_window (app, window);
+  g_signal_handlers_disconnect_by_func (window, G_CALLBACK(on_wm_class_changed), self);
 
   g_signal_emit (self, signals[TRACKED_WINDOWS_CHANGED], 0);
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]