[Bug 34834] New: [Bisected] Thief: Deadly Shadows regression: Player cannot turn around freely in fullscreen mode

http://bugs.winehq.org/show_bug.cgi?id=34834

Bug #: 34834
           Summary: [Bisected] Thief: Deadly Shadows regression: Player
                    cannot turn around freely in fullscreen mode
           Product: Wine
           Version: unspecified
          Platform: x86
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified
   Regression SHA1: 76bbf106a28c4caa82873e8450bde7d4adc765bf

When running this game in fullscreen it is not possible for the player to spin
around freely. This is a regression between wine 1.4 and 1.6, still present.

Steps to reproduce:

- The free demo behaves the same as the actual game and can be used for
testing: http://www.gamefront.com/files/3090442
- Change your desktop resolution to 1600x1200
- Run the game. (The default path for the demo is C:\Program Files\Thief -
Deadly Shadows Demo\System\t3.exe)
  Press ESC several times to skip movies. From the menu go to the options
screen and change the game resolution to 1600x1200
- Quit the game. If the buttons have become inaccessible after the resolution
change, then press Alt-Enter
- Run the game again. From the menu click New Game, then Start, then (after
loading) OK
- Move the mouse to one side and try to spin on the spot (look straight up to
make it easier to see). You will find that your movement is restricted
- Press Alt-Enter to go to windowed mode, click on the window and try again.
Now you can turn freely

Notes:

- Apparently the desktop resolution must match the game resolution for
fullscreen to work properly.
- If the game menu behaves weirdly, try running the game on a single core
- On Ubuntu/Unity it may be necessary to enable Legacy Fullscreen Support in
Compiz (but not in my case)

Bisection from 1.4 to 1.6 found this commit:
> 76bbf106a28c4caa82873e8450bde7d4adc765bf is the first bad commit
> commit 76bbf106a28c4caa82873e8450bde7d4adc765bf
> Author: Henri Verbeet 
> Date:   Thu Apr 25 09:59:42 2013 +0200
> winex11: Move the desktop process check from X11DRV_ClipCursor() to grab_clipping_window().
>
> We don't want to clip in the desktop process, but we still need it to call
> ungrab_clipping_window() if the process that was previously clipping didn't.
> This can happen for example when fullscreen clipping is enabled, but the
> corresponding window isn't explicitly destroyed before process exit.
>
> :040000 040000 d6362b9803969c01c9c86d739c6d8f958ad1866c ea562a8bc473131d579f51c59e7bff18809f6504 M	dlls

Tested with a GT430 card and the proprietary NVIDIA driver in Ubuntu 13.10 and
from a Fedora live-CD. (Many graphics issues with the Nouveau driver but it
could be verified that the behaviour was the same in fullscreen and windowed
mode.