<Swing Dev> [14] RFR JDK-8234149:Several regression tests do not dispose Frame at end
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Mon Dec 2 04:48:12 UTC 2019
Looks fine.
On 11/24/19 10:17 pm, Prasanta Sadhukhan wrote:
> Hi Sergey,
>
> Thanks for your time. I have removed HeadlessJFrame and added @hey headful to bug4796987.java
>
> http://cr.openjdk.java.net/~psadhukhan/8234149/webrev.2/
>
> Regards
>
> Prasanta
>
> On 23-Nov-19 5:30 AM, Sergey Bylokhov wrote:
>> Hi, Prasanta.
>>
>> Two small comments:
>>
>> - The changes in "test/jdk/javax/swing/Headless/HeadlessJFrame.java" are not necessary
>> the frame never becomes visible or packed, no need to dispose of i
>> (I guess it is applicable to all tests which do not have "@key headful" tag).
>>
>> - @key headful should be added to "test/jdk/javax/swing/JButton/4796987/bug4796987.java"
>>
>>
>> On 11/22/19 1:30 am, Prasanta Sadhukhan wrote:
>>> ping? any more feedback on this?
>>>
>>> On 18-Nov-19 4:18 PM, Prasanta Sadhukhan wrote:
>>>> I have removed the manual test updation. Rest are passing and mach5 is green (except many nothing to run tests which are either problemlisted or platform-specific)
>>>>
>>>> http://cr.openjdk.java.net/~psadhukhan/8234149/webrev.1/
>>>>
>>>> Regards
>>>>
>>>> Prasanta
>>>>
>>>> On 18-Nov-19 9:53 AM, Sergey Bylokhov wrote:
>>>>> On 11/14/19 11:02 pm, Prasanta Sadhukhan wrote:
>>>>>> Hi Sergey,
>>>>>>
>>>>>> I can remove the dispose from manual tests, but are you questioning the basic need of these dispose in "all" tests?
>>>>>
>>>>> Only about manual tests.
>>>>>
>>>>>>
>>>>>> On 15-Nov-19 12:17 PM, Sergey Bylokhov wrote:
>>>>>>> Hi, Prasanta.
>>>>>>>
>>>>>>> In a few files you have updated the manual tests which I think changed the behavior of the tests, for example:
>>>>>>> test/jdk/javax/swing/JSlider/6524424/bug6524424.java
>>>>>>>
>>>>>>> Not sure that such code is useful:
>>>>>>> 59 frame.setVisible(true);
>>>>>>> 60 } finally {
>>>>>>> 61 if (frame != null) SwingUtilities.invokeAndWait(() -> frame.dispose());
>>>>>>> 62 }
>>>>>>>
>>>>>>> On 11/14/19 12:56 am, Prasanta Sadhukhan wrote:
>>>>>>>> Hi All,
>>>>>>>>
>>>>>>>> Please review a fix for an issue seen in several tests where the JFrame created is not disposed of at all code paths.
>>>>>>>>
>>>>>>>> This sometimes leaves some artifacts thereby affecting execution of subsequent tests, causing those to fail.
>>>>>>>>
>>>>>>>> Proposed fix is to make sure the JFrame is disposed in try-finally block to ensure frame is disposed irrespective of execution outcome of the test.
>>>>>>>>
>>>>>>>> This list is not exhaustive as there are still some tests it is not covered here and
>>>>>>>>
>>>>>>>> I have not taken care of indentation to ensure only the change/modification is highlighted and not all indented lines.
>>>>>>>>
>>>>>>>> Mach5 status link is there in JBS.
>>>>>>>>
>>>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8234149
>>>>>>>>
>>>>>>>> webrev: http://cr.openjdk.java.net/~psadhukhan/8234149/webrev.0/
>>>>>>>>
>>>>>>>> Regards
>>>>>>>>
>>>>>>>> Prasanta
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>
>>>>>
>>
>>
--
Best regards, Sergey.
More information about the swing-dev
mailing list