[regression] "Remember Focus" does not play well with unity launcher so shouldn't be default enabled or better yet the rev should be reverted

I'm having trouble reproducing this on an upgrade Vivid install: when I click the Nautilus icon in the launcher I'm taken back to WS1 and the Nautilus window is in focus (all keyboard input is directed there).

Hmm, a short vid attached -
Starts with nautilus & a terminal open, focus on terminal. Switching to ws2 & clicking on nautilus icon returns view to ws1 with nautilus on top but focus remains on terminal.

Then putting focus on nautilus > go to ws2, clicking on terminal icon returns to ws1 with terminal on top but focus remains on nautilus.

I think the real point here is introducing an option that changes expected behavior with no user method to change other than thru ccsm

Status changed to 'Confirmed' because the bug affects multiple users.

I also have this kind of problem.

But I also note this other problem which is not reproducible each time but I guess this bug is related to this one:
* Open two maximised windows: one on the first workspace, the other on the second one.
* Switch between workspaces and the focus will not changed.
e.g. I have my text editor on workspace 1 and Firefox on the next workspace. My text editor's window has the focus and it keeps it even if I switch to the next workspace.

I didn't have this bug on Ubuntu 14.10.

As matttbe has mentioned there are other issues.
Have attached very short vid to demonstrate -
On ws1 there are 2l open windows, focus on nautilus, on ws2 there is a firefox window only

From ws1 clicking on FF icon goes to ws2 but focus remains on nautilus window on ws1
From ws2 clicking on the nautilus icon returns to ws1 but focus remains on FF on ws2
ect.

Actually one has to wonder what was the point of this change. No bug was mentioned in changelog, as a feature it has no value.
So it should either be fixed (to what purpose??) or instead of disabling by default the change should be reverted completely

Please note that the above comment's issue *may* only be when using rotate (orig. issue occurs with both wall & rotate)

I see the commit that caused this was a bug fix for https://bugs.launchpad.net/compiz/0.9.11/+bug/1125442
It's probably better when doing something like this to note the bug in the changelog rather than generic entry -

[ Dariusz Gadomski ]
  * CompScreen: Save focused window id before changing viewport.

I see a new proposed commit which seems to fix the ^ but not too sure it may not cause issues with rotate users
(- if that's a concern, (rotate),, then may be good to give a reasonable test before releasing

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