That looks fine, although modifying the passed descriptor is a little surprising. I'm less familiar with d3d9 conventions, so if @zfigura approves, I'm not objecting, but maybe we could only change the descriptor stored in `struct wined3d_swapchain_state`, and if it needs to be updated up the stack, call `wined3d_swapchain_get_desc`?