Re: wined3d/d3d9: Set the initial scissorrect to the dimesions of the backbuffer from the first swapchain of the device

[Andrew Fenn]

This works nicely for me. Spore displays correctly now with this patch
and latest git. I also tried your test case on XP and it passed.

On Sun, Aug 24, 2008 at 1:09 AM, Rico Schüller  wrote:
> Hi,
>
> this is the second try for this patch. I've renamed it because the previous
> description was wrong.
>
> This patch adds a test for the initial scissorrect values and fixes the
> wrong behaviour in wine.
>
> This solves partly bug 13988.
>
> Cheers
> Rico
>
> From 8f72f697284122cd3d691dc8927b127a888f6bde Mon Sep 17 00:00:00 2001
> From: =?utf-8?q?Rico=20Sch=C3=BCller?= 
> Date: Sat, 23 Aug 2008 19:59:39 +0200
> Subject: [PATCH] wined3d/d3d9: Set the initial scissorrect to the dimesions
> of the backbuffer from the first swapchain of the device
>
> dlls/d3d9/tests/device.c  |  104
> +++++++++++++++++++++++++++++++++++++++++++++
>  dlls/wined3d/stateblock.c |   21 +++++++++
>  2 files changed, 125 insertions(+), 0 deletions(-)
>
> diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c
> index 3164b81..a430248 100644
> --- a/dlls/d3d9/tests/device.c
> +++ b/dlls/d3d9/tests/device.c
> @@ -2016,6 +2016,109 @@ static void test_display_formats()
>     if(d3d9) IDirect3D9_Release(d3d9);
>  }
>
> +static void test_scissor_size(void)
> +{
> - IDirect3D9 *d3d9_ptr = 0;
> - int i;
> - static const struct {
> -
>    int winx; int winy; int backx; int backy; int backswap1x; int
>
> backswap1y; BOOL window;
> - } scts[] = { /* scissor tests */
> -
>    {800, 600, 640, 480, 1024, 768, TRUE},
> -
>    {800, 600, 640, 480, 1024, 768, FALSE},
> -
>    {640, 480, 800, 600, 1024, 768, TRUE},
> -
>    {640, 480, 800, 600, 1024, 768, FALSE},
> -
> -
>    {1024, 768, 640, 480, 800, 600, TRUE},
> -
>    {1024, 768, 640, 480, 800, 600, FALSE},
> -
>    {640, 480, 1024, 768, 800, 600, TRUE},
> -
>    {640, 480, 1024, 768, 800, 600, FALSE},
> -
> -
>    {1024, 768, 800, 600, 640, 480, TRUE},
> -
>    {1024, 768, 800, 600, 640, 480, FALSE},
> -
>    {800, 600, 1024, 768, 640, 480, TRUE},
> -
>    {800, 600, 1024, 768, 640, 480, FALSE},
> - };
> -
> - d3d9_ptr = pDirect3DCreate9(D3D_SDK_VERSION);
> - ok(d3d9_ptr != NULL, "Failed to create IDirect3D9 object\n");
> - if (!d3d9_ptr){
> -
>    skip("Failed to create IDirect3D9 object\n");
> -
>    return;
> - }
> -
> - for(i=0; i<sizeof(scts)/sizeof(scts[0]); i++) {
> -
>    IDirect3DDevice9 *device_ptr = 0;
> -
>    D3DPRESENT_PARAMETERS present_parameters;
> -
>    HRESULT hr;
> -
>    WNDCLASS wc = {0};
> -
>    HWND hwnd = 0;
> -
>    IDirect3DSwapChain9 *swapchain1 = 0;
> -
>    RECT scissorrect;
> -
> -
>    wc.lpfnWndProc = DefWindowProc;
> -
>    wc.lpszClassName = "d3d9_test_wc";
> -
>    RegisterClass(&wc);
> -
> -
>    hwnd = CreateWindow("d3d9_test_wc", "d3d9_test",
> -
>                    WS_MAXIMIZE | WS_VISIBLE | WS_CAPTION , 0, 0,
>
> scts[i].winx, scts[i].winy, 0, 0, 0, 0);
> -
> -
>    ZeroMemory(&present_parameters, sizeof(present_parameters));
> -
>    present_parameters.Windowed = scts[i].window;
> -
>    present_parameters.hDeviceWindow = hwnd;
> -
>    present_parameters.SwapEffect = D3DSWAPEFFECT_DISCARD;
> -
>    present_parameters.BackBufferWidth = scts[i].backx;
> -
>    present_parameters.BackBufferHeight = scts[i].backy;
> -
>    present_parameters.BackBufferFormat = D3DFMT_A8R8G8B8;
> -
>    present_parameters.EnableAutoDepthStencil = TRUE;
> -
>    present_parameters.AutoDepthStencilFormat = D3DFMT_D24S8;
> -
> -
>    hr = IDirect3D9_CreateDevice(d3d9_ptr, D3DADAPTER_DEFAULT,
>
> D3DDEVTYPE_HAL, present_parameters.hDeviceWindow,
> D3DCREATE_HARDWARE_VERTEXPROCESSING, &present_parameters, &device_ptr);
> -
>    if(FAILED(hr)) {
> -
>        present_parameters.AutoDepthStencilFormat = D3DFMT_D16;
> -
>        hr = IDirect3D9_CreateDevice(d3d9_ptr, D3DADAPTER_DEFAULT,
>
> D3DDEVTYPE_HAL, present_parameters.hDeviceWindow,
> D3DCREATE_HARDWARE_VERTEXPROCESSING, &present_parameters, &device_ptr);
> -
>        if(FAILED(hr)) {
> -
>            hr = IDirect3D9_CreateDevice(d3d9_ptr, D3DADAPTER_DEFAULT,
>
> D3DDEVTYPE_HAL, present_parameters.hDeviceWindow,
> D3DCREATE_SOFTWARE_VERTEXPROCESSING, &present_parameters, &device_ptr);
> -
>        }
> -
>    }
> -
>    ok(hr == D3D_OK || hr == D3DERR_NOTAVAILABLE,
>
> "IDirect3D_CreateDevice returned: %08x\n", hr);
> -
> -
>    if (!device_ptr)
> -
>    {
> -
>        DestroyWindow(hwnd);
> -
>        skip("Creating the device failed\n");
> -
>        goto err_out;
> -
>    }
> -
> -
>    /* add a swapchain only in window mode */
> -
>    if(scts[i].window)
> -
>    {
> -
>        present_parameters.BackBufferWidth = scts[i].backswap1x;
> -
>        present_parameters.BackBufferHeight = scts[i].backswap1y;
> -
>        hr = IDirect3DDevice9_CreateAdditionalSwapChain(device_ptr,
>
> &present_parameters, &swapchain1);
> -
>        ok(hr == D3D_OK, "IDirect3DDevice9_CreateAdditionalSwapChain
>
> failed in test %i with: %08x\n", i, hr);
> -
>    }
> -
> -
>    /* Check for the default scissor rect size */
> -
>    hr = IDirect3DDevice9_GetScissorRect(device_ptr, &scissorrect);
> -
>    ok(hr == D3D_OK, "IDirect3DDevice9_GetScissorRect failed with:
>
> %08x\n", hr);
> -
>    ok(scissorrect.right == scts[i].backx && scissorrect.bottom ==
>
> scts[i].backy && scissorrect.top == 0 && scissorrect.left == 0, "Scissorrect
> missmatch (%d, %d) should be (%d, %d)\n", scissorrect.right,
> scissorrect.bottom, scts[i].backx, scts[i].backy);
> -
> -
>    if(swapchain1) IDirect3DSwapChain9_Release(swapchain1);
> -
>    if(device_ptr) {
> -
>        ULONG ref;
> -
> -
>        ref = IDirect3DDevice9_Release(device_ptr);
> -
>        DestroyWindow(hwnd);
> -
>        ok(ref == 0, "The device was not properly freed: refcount
>
> %u\n", ref);
> -
>    }
> - }
> -
>
> +err_out:
> - if(d3d9_ptr) IDirect3D9_Release(d3d9_ptr);
> - return;
>
> +}
> -
> -
>
> START_TEST(device)
>  {
>     HMODULE d3d9_handle = LoadLibraryA( "d3d9.dll" );
> @@ -2045,5 +2148,6 @@ START_TEST(device)
>         test_vertex_buffer_alignment();
>         test_lights();
>         test_set_stream_source();
> -
>    test_scissor_size();
> }
>
> }
> diff --git a/dlls/wined3d/stateblock.c b/dlls/wined3d/stateblock.c
> index e8ff6c1..e953470 100644
> --- a/dlls/wined3d/stateblock.c
> +++ b/dlls/wined3d/stateblock.c
> @@ -1037,6 +1037,11 @@ static HRESULT  WINAPI
> IWineD3DStateBlockImpl_InitStartupStateBlock(IWineD3DStat
>         DWORD d;
>     } tmpfloat;
>     unsigned int i;
> -
>
> IWineD3DSwapChain *swapchain;
> -
>
> IWineD3DSurface *backbuffer;
> -
>
> WINED3DSURFACE_DESC desc = {0};
> -
>
> UINT width, height;
> -
>
> RECT scissorrect;
>
> /* Note this may have a large overhead but it should only be executed
>   once, in order to initialize the complete state of the device and
>
> @@ -1239,6 +1244,22 @@ static HRESULT  WINAPI
> IWineD3DStateBlockImpl_InitStartupStateBlock(IWineD3DStat
>         This->textures[i]         = NULL;
>     }
> - /* Set the default scissor rect values */
> - desc.Width = &width;
> - desc.Height = &height;
> -
> - IWineD3DDevice_GetSwapChain(device, 0, &swapchain);
> - IWineD3DSwapChain_GetBackBuffer(swapchain, 0,
>
> WINED3DBACKBUFFER_TYPE_MONO, &backbuffer);
> - IWineD3DSurface_GetDesc(backbuffer, &desc);
> - IWineD3DSurface_Release(backbuffer);
> - IWineD3DSwapChain_Release(swapchain);
> -
> - scissorrect.left = 0;
> - scissorrect.right = width;
> - scissorrect.top = 0;
> - scissorrect.bottom = height;
> - IWineD3DDevice_SetScissorRect(device, &scissorrect);
> - TRACE("-----------------------> Device defaults now set up...\n");
>   return WINED3D_OK;
> }
>
> 1.5.5.1