[regression] Multiple viewport switches in one movement causes incorrect window focusing.

Bug #1398512 reported by Brandon Schaefer on 2014-12-02
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
High
Brandon Schaefer
compiz (Ubuntu)
High
Brandon Schaefer

Bug Description

This bug started after this merge:
https://code.launchpad.net/~dgadomski/compiz/remember-focus/+merge/236735

How to reproduce:
TL = Top Left workspace
TR = Top Right
BL = Bottom Left
BR = Bottom Right

1) Open a window on TL, TR, and BL
2) Move to BL workspace.
3) Quickly move to TR workspace, moving through BR or TL workspace.
4) Now move to BR/TL workspace.

Expected Result:
-- Last active window on that workspace gets window focus

Actual Result:
-- Window that was LAST ACTIVE (ie. the window in the BL workspace) gets focus.

This happens because the Window focus does not change when moving through workspaces, but new MoveWorkspaces function calls happen. So when you move through a workspace we are saving the active window as the last active window for that workspace.

Related branches

Changed in compiz:
status: New → Triaged
importance: Undecided → High
importance: High → Medium
Changed in compiz (Ubuntu):
importance: Undecided → Critical
importance: Critical → Medium
status: New → In Progress
Changed in compiz:
status: Triaged → In Progress
assignee: nobody → Brandon Schaefer (brandontschaefer)
Changed in compiz (Ubuntu):
assignee: nobody → Brandon Schaefer (brandontschaefer)
Changed in compiz:
importance: Medium → Critical
summary: - Multiple viewport switches in one movement causes incorrect window
- focusing.
+ [regression] Multiple viewport switches in one movement causes incorrect
+ window focusing.
Changed in compiz (Ubuntu):
importance: Medium → Critical
Changed in compiz:
importance: Critical → High
Changed in compiz (Ubuntu):
importance: Critical → High

Revert has been merged, so this bug no longer exists.

Changed in compiz:
status: In Progress → Fix Committed
Changed in compiz (Ubuntu):
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package compiz - 1:0.9.12.0+15.04.20141210.2-0ubuntu1

---------------
compiz (1:0.9.12.0+15.04.20141210.2-0ubuntu1) vivid; urgency=medium

  [ James Jones ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Kyle Brenneman ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Viktor A. Danilov ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Chris Townsend ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Stephen M. Webb ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)
  * libdecoration/CMakeLists.txt: use correct variable for link
    directories (LP: #1392715)
  * Switcher Plugin: renamed a local variable so it does not hide a
    function parameter. (LP: #1101585)
  * added support for multi-arch installations (LP: #1395105, #959282)
  * libcompizconfig: reorder stat() and open() calls to eliminate a race
    condition (LP: #1101608)
  * dodge plugin: avoid potential null pointer dereferences (LP:
    #1101554)
  * PrivateScreen unit tests: added additional assertions to verify the
    tests are run in an expected state (LP: #1101413)

  [ Ubuntu daily release ]
  * New rebuild forced

  [ Brandon Schaefer ]
  * Rev 3891 change caused 2 known regressions. (Could be more).
    Reverting. (LP: #1398512, #1393020)
 -- Ubuntu daily release <email address hidden> Wed, 10 Dec 2014 19:52:06 +0000

Changed in compiz (Ubuntu):
status: Fix Committed → Fix Released
Stephen M. Webb (bregma) on 2015-03-19
Changed in compiz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers