wined3d: Allow SetCursorProperties on existing cursor

[Stefan Dösinger]

> The API documentation doesn't say anything about an application releasing
> the d3d surface,
There is a test for that somewhere in dlls/d3d9/tests/device.c (or some other 
test file in there).

> but the documentation in the code seems to indicate that 
> the gl texture needs to remain when the surface is deleted - I did nothing
> to change this. 
But IWineD3DSurfaceImpl_Release will destroy the gl texture of a surface when 
the surface is destroyed.

Also, if the gl surface still knows the texture, the following could cause an 
issue:

SetCursorProperties(cursorSurface, ...);

<render some stuff>

LockRect(cursorSurface, ...);
<write some thing into the surface>
UnlockRect(cursorSurface);
PreLoad(cursorSurface);

The PreLoad will then change the cursor without SetCursorProperties beeing 
called. This should not happen. I tested that with some hacky test. 
Unfortunately this cannot be automated in the visual tests because cursors do 
not show up in screenshots.

This is my hacky cursor test app:
http://stud4.tuwien.ac.at/~e0526822/visual.c

The successor of that file without the cursor tests is now in 
dlls/d3d9/tests/visual.c. I think this visual.c version is not the version I 
used for checking later surface changes.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20070311/1cd4fef9/attachment.pgp