Fix for focus loops

[Jukka Heinonen]

On Sun, May 12, Alexandre Julliard wrote:
> Actually all the problems you list apply only to unmanaged
> mode. Fixing that mode to be ICCCM compliant is fairly low priority,
> and probably quite hopeless. In fact the whole concept of unmanaged
> mode is flawed, and it will probably be replaced by a
> managed-with-no-decorations mode (with a few exceptions for menus
> etc.) once we have the proper support for doing actions like
> min/maximize through the window manager.

I'm sorry to sound disrespectful, but I believe that many of those 
problems I have listed do apply to managed mode, at least if 
decorations are drawn by Wine. Think about the following scenario:

- User has set window manager into mode where focus is not
  automatically given to new windows (can be done at least with
  WindowMaker).
- User starts Wine program from xterm command line.
- New Wine window is opened, Wine activates window, calls
  X11DRV_SetFocus and draws active caption bar.
  However, this is wrong since window manager has kept focus 
  on xterm window and Wine should have drawn passive caption bar.

You are partially correct in the fact that this kind of problems
are unlikely to cause serious problems in managed mode. However, 
they still are bugs, potentially quite visible bugs, too, depending
on what (ICCCM compliant) window manager is being used and how 
that window manager has been configured.

-- 
Jukka Heinonen <http://www.iki.fi/jhei/>