On Tue, 12 Oct 2021 06:27:14 GMT, Sergey Bylokhov <[email protected]> wrote:
>> Could you please review the 7001973 fixes? >> >> On Windows, CopyAreaOOB.java fails with a blank window. The cause of this >> problem is that paint() works one time only. Painting area is not guaranteed >> when showing a window. >> >> I think this behavior should be removed for purpose of this test (`@summary: >> Verifies that copyArea() works properly`). Also, this program uses a Robot, >> but implements waiting logic by itself. This logic should be replaced to >> Robot API. >> >> This fix works fine in Windows, so this test can be removed from >> ProblemList.txt. I don't have a Mac environment, so can someone please >> confirm that this fix will work on Mac? > > You will not be able to write the test which will check that only one paint > event is posted, it is not guaranteed. > You can only check that each rendering caused by the paint event will work > fine. This test case check that the first event works fine, it can be > improved to check all paint events. > > So for the test update, you can add a big delay after the rendering, and wait > for what will occur on the screen, any ideas why it may not work? You also > can create a separate bug to investigate how we can validate the surface > before showing the window so we did not post more events than necessary. @mrserb Could you please reply to the above comments and fix of test? ------------- PR: https://git.openjdk.java.net/jdk/pull/5491
