<Swing Dev> [14] RFR JDK-8234149:Several regression tests do not dispose Frame at end

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Mon Nov 25 06:17:16 UTC 2019


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
>>>>>>>
>>>>>>
>>>>>>
>>>>
>>>>
>
>


More information about the swing-dev mailing list