Merge lp:~3v1n0/unity/quick-alt+tab-fixes into lp:unity

[Marco Trevisan (Treviño)]

Make the quick Alt+Tab more consistent.

The quick Alt+Tab should be handled in a different way according to bug #861250, so I've made it possible by making the SwitcherController to use the ActionArg button member to define the type of the switch that has been performed (I've made this not to add an extra member to the class, but this can be changed if you want the code to be more clean).
Then the activated BamfLauncherIcon redirects this information to the PluginAdapter's FocusWindowGroup that now when called with the FocusVisibility::OnlyVisibleOnTop visibility option, just brings to focus the first window that is on top of the stack (for the given application).

A manual test is included.

[Tim Penhey (thumper)]

How does this interaction work with the alt-tab switcher defaulting to only showing applications running on the same workspace? Are there any considerations we have to make there?

review: Needs Information

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

Mh, in that case this modification doesn't affect the switcher behavior.

I've to look better to this case, to see how we can handle it correctly.

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

> How does this interaction work with the alt-tab switcher defaulting to only
> showing applications running on the same workspace? Are there any
> considerations we have to make there?

I've gave a look to this, but in this case it's the switcher itself that defines the possible windows to switch to.
So, In that case, my code is used with only a subset of windows, and so it should continue to work as expected.

=== added file 'manual-tests/QuickAltTab.txt'
--- manual-tests/QuickAltTab.txt	1970-01-01 00:00:00 +0000
+++ manual-tests/QuickAltTab.txt	2011-12-15 03:29:25 +0000
@@ -0,0 +1,17 @@
+Quick Alt+Tab
+-------------
+This test shows how the alt+tab works when pressed quickly
+
+#. Start on a clean screen
+#. Open two windows of the same application (i.e. gnome-terminal)
+#. Open another window of another application (i.e. gedit)
+#. Focus a gnome-terminal window
+#. Quickly press Alt+Tab
+#. The gedit window should be now on focus
+#. Press again Alt+Tab
+
+Outcome
+  The gnome-terminal window that had been in focus before should be brought to
+  focus again. The second gnome-terminal window should below the gnome-terminal
+  and gedit windows. The same should happen if the gnome-terminal windows are
+  in different workspaces.
 
=== modified file 'plugins/unityshell/src/BamfLauncherIcon.cpp'
--- plugins/unityshell/src/BamfLauncherIcon.cpp	2011-12-13 08:58:59 +0000
+++ plugins/unityshell/src/BamfLauncherIcon.cpp	2011-12-15 03:29:25 +0000
@@ -295,8 +295,10 @@
     {
       guint32 xid = bamf_window_get_xid(BAMF_WINDOW(view));
 
-      if (wm->IsWindowMapped(xid))
-        results.push_back ((Window) xid);
+      if (wm->IsWindowMapped(xid) && !bamf_window_get_transient(BAMF_WINDOW(view)))
+      {
+        results.push_back (xid);
+      }
     }
   }
 
@@ -628,8 +630,18 @@
     }
   }
   else
-    WindowManager::Default()->FocusWindowGroup(windows,
-      WindowManager::FocusVisibility::OnlyVisible);
+  {
+    if (arg.button == 0)
+    {
+      WindowManager::Default()->FocusWindowGroup(windows,
+        WindowManager::FocusVisibility::OnlyVisible);
+    }
+    else
+    {
+      WindowManager::Default()->FocusWindowGroup(windows,
+        WindowManager::FocusVisibility::OnlyVisibleOnTop);
+    }
+  }
 }
 
 bool BamfLauncherIcon::Spread(int state, bool force)
 
=== modified file 'plugins/unityshell/src/PluginAdapter.cpp'
--- plugins/unityshell/src/PluginAdapter.cpp	2011-10-14 09:46:14 +0000
+++ plugins/unityshell/src/PluginAdapter.cpp	2011-12-15 03:29:25 +0000
@@ -568,7 +568,7 @@
     window->lower();
 }
 
-void 
+void
 PluginAdapter::FocusWindowGroup(std::vector<Window> window_ids, FocusVisibility focus_visibility)
 {
   CompPoint target_vp = m_Screen->vp();
@@ -603,50 +603,63 @@
       break;
   }
 
-  if (!any_on_current)
+  if (!any_on_current || focus_visibility == FocusVisibility::OnlyVisibleOnTop)
   {
-    for (auto it = windows.rbegin(); it != windows.rend(); it++)
+    for (auto it = windows.rbegin(); it != windows.rend(); ++it)
     {
-      if ((any_mapped && !(*it)->minimized()) || !any_mapped)
+      CompWindow* win = *it;
+      if ((any_mapped && !win->minimized()) || !any_mapped)
       {
-        target_vp = (*it)->defaultViewport();
+        if (focus_visibility == FocusVisibility::OnlyVisibleOnTop)
+        {
+          win->raise();
+          top_win = win;
+        }
+        else
+        {
+          target_vp = win->defaultViewport();
+        }
+
         break;
       }
     }
   }
 
-  for (CompWindow* &win : windows)
+  if (focus_visibility != FocusVisibility::OnlyVisibleOnTop)
   {
-    if (win->defaultViewport() == target_vp)
+    for (CompWindow* &win : windows)
     {
-       /* Any window which is actually unmapped is
+      if (win->defaultViewport() == target_vp)
+      {
+        /* Any window which is actually unmapped is
         * not going to be accessible by either switcher
         * or scale, so unconditionally unminimize those
         * windows when the launcher icon is activated */
-       if ((focus_visibility == WindowManager::FocusVisibility::ForceUnminimizeOnCurrentDesktop &&
-            WindowManager::Default ()->IsWindowOnCurrentDesktop(win->id ())) ||
-            (focus_visibility == WindowManager::FocusVisibility::ForceUnminimizeInvisible &&
+        if ((focus_visibility == FocusVisibility::ForceUnminimizeOnCurrentDesktop &&
+             WindowManager::Default()->IsWindowOnCurrentDesktop(win->id())) ||
+            (focus_visibility == FocusVisibility::ForceUnminimizeInvisible &&
              win->mapNum () == 0))
-       {
-         bool is_mapped = win->mapNum () != 0;
-         top_win = win;
-         win->unminimize ();
-
-         forced_unminimize = true;
-
-         /* Initially minimized windows dont get raised */
-         if (!is_mapped)
-           win->raise ();
-       }
-       else if ((any_mapped && !win->minimized()) || !any_mapped)
-       {
-         if (!forced_unminimize ||
-             WindowManager::Default ()->IsWindowOnCurrentDesktop (win->id ()))
-         {
-           win->raise();
-           top_win = win;
-         }
-       }
+        {
+          bool is_mapped = win->mapNum() != 0;
+          win->unminimize();
+          top_win = win;
+
+          forced_unminimize = true;
+
+          /* Initially minimized windows dont get raised */
+          if (!is_mapped)
+            win->raise();
+        }
+        else if ((any_mapped && !win->minimized()) || !any_mapped)
+        {
+          if (!forced_unminimize ||
+              WindowManager::Default()->IsWindowOnCurrentDesktop(win->id()))
+          {
+            win->raise();
+            top_win = win;
+          }
+        }
+      }
     }
   }
 
 
=== modified file 'plugins/unityshell/src/SwitcherController.cpp'
--- plugins/unityshell/src/SwitcherController.cpp	2011-12-14 16:18:41 +0000
+++ plugins/unityshell/src/SwitcherController.cpp	2011-12-15 03:29:25 +0000
@@ -205,20 +205,25 @@
     AbstractLauncherIcon* selection = model_->Selection();
     if (selection)
     {
+      /* Instead of defining a new type of ActionArg we only use the "button"
+       * ActionArg value to indicate to the view if we're performing a quick
+       * switch or not.                                                      */
+      unsigned int is_quick = (view_window_ && view_window_->IsVisible()) ? 0 : 1;
+
       if (model_->detail_selection)
       {
-        selection->Activate(ActionArg(ActionArg::SWITCHER, 0, model_->DetailSelectionWindow ()));
+        selection->Activate(ActionArg(ActionArg::SWITCHER, is_quick, model_->DetailSelectionWindow ()));
       }
       else
       {
         if (selection->GetQuirk (AbstractLauncherIcon::QUIRK_ACTIVE) &&
             !model_->DetailXids().empty ())
         {
-          selection->Activate(ActionArg (ActionArg::SWITCHER, 0, model_->DetailXids()[0]));
+          selection->Activate(ActionArg (ActionArg::SWITCHER, is_quick, model_->DetailXids()[0]));
         }
         else
         {
-          selection->Activate(ActionArg(ActionArg::SWITCHER, 0));
+          selection->Activate(ActionArg(ActionArg::SWITCHER, is_quick));
         }
       }
     }
 
=== modified file 'plugins/unityshell/src/WindowManager.h'
--- plugins/unityshell/src/WindowManager.h	2011-10-14 09:18:55 +0000
+++ plugins/unityshell/src/WindowManager.h	2011-12-15 03:29:25 +0000
@@ -48,6 +48,7 @@
   enum class FocusVisibility
   {
     OnlyVisible,
+    OnlyVisibleOnTop,
     ForceUnminimizeInvisible,
     ForceUnminimizeOnCurrentDesktop
   };