"present_rect" is a bit of an unfortunate name, since that could refer to one or more of the rects passed to wined3d_swapchain_present(). "drawable_rect" would be more in line with existing terminology.
Well I don't really know about the wined3d terminology, and I'm fine to change it to any name on that side of things but it does seem more confusing to me at least for the win32u side: `window drawable rect` could be read as "the window rect that can be drawn to", and this is not what this is. The rect is not a rectangle inside the window, and not really something that can be drawn to from the window perspective, but rather the area of the screen the window swapchain images should be presented to. Then I would think this to be better to have consistent names across modules but I'm fine renaming `set_window_present_rect` to `set_window_drawable_rect` if that is better for you. (Also fwiw, I think `present` makes it close to D3DKMTPresent which this is fairly related to)
The upshot of this is that the whole of get_swapchain_present_rect() is pretty much unnecessary; you can just use the swapchain size directly.
Thanks, I changed the code to use swapchain backbuffer width/height, I understand this is what you meant? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9585#note_124757