RFR: 8185862: AWT Assertion Failure in ::GetDIBits(hBMDC, hBM, 0, 1, 0, gpBitmapInfo, 0) 'awt_Win32GraphicsDevice.cpp', at line 185 [v9]

[Alexey Ivanov]

On Thu, 7 Mar 2024 16:48:27 GMT, Christoph Langer <clanger at openjdk.org> wrote:

>> If you don't add the assertions for `hBMDC` and `hBM`, you should revert all the changes to `awt_Win32GraphicsDevice.cpp`.
>> 
>> At the moment, two lines are removed from `initScreens`.
>
> The two lines removed from initScreens are in `awt_Win32GraphicsEnv.cpp` 😉  That's unrelated cleanup I guess. But I bring back the VERIFY code which seems to make sense and would not alter the behavior in the non-debug case.

You're right! `awt_Win32GraphicsDevice.cpp` is not part of the PR at this very moment.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17614#discussion_r1516508137