<Swing Dev> RFR: 8261689: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" [v2]
Alexander Zuev
kizune at openjdk.java.net
Wed Mar 3 05:09:38 UTC 2021
On Tue, 2 Mar 2021 23:17:26 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Well, functionally it will not make any difference, we are analyzing the screenshots and the only thing that matters is at which time screenshot was taken. The only reason we don't take all screenshots in the beginning and analyze them later is to save test run time if test runs early. At the end it is not critical anymore. And as i said this is an existing test so i'm trying not to change it unless it affects its functionality.
>
> Yes, I agree it makes no difference. Yet when you read the test code, the pattern — show or hide the button, take screenshot, analyze it — gets broken. This place is the only exception to the pattern; here the button is shown, screenshot is taken, then the button is hidden and only after hiding the button the screenshot is analyzed, which could give the wrong impression as if the screenshot after hiding the button is being analyzed here.
Ok, i give up, you won - i'll fix it.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2790
More information about the swing-dev
mailing list