<Swing Dev> [12] RFR JDK-8203904:javax/swing/JSplitPane/4816114/bug4816114.java: The divider location is wrong

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Tue Jul 31 22:36:57 UTC 2018


Looks fine.

On 27/07/2018 07:57, Prasanta Sadhukhan wrote:
> Can I have a 2nd "R"eviewer for this?
> 
> Regards
> Prasanta
> On 7/25/2018 4:59 PM, Krishna Addepalli wrote:
>> +1
>>
>> -----Original Message-----
>> From: Prasanta Sadhukhan
>> Sent: Wednesday, July 25, 2018 11:59 AM
>> To: Krishna Addepalli <krishna.addepalli at oracle.com>; 
>> swing-dev at openjdk.java.net
>> Subject: Re: <Swing Dev> [12] RFR 
>> JDK-8203904:javax/swing/JSplitPane/4816114/bug4816114.java: The 
>> divider location is wrong
>>
>> Thanks Krishna for the review.
>>
>> Here's the modified webrev
>> http://cr.openjdk.java.net/~psadhukhan/8203904/webrev.1/
>>
>> Regards
>> Prasanta
>> On 7/24/2018 10:10 PM, Krishna Addepalli wrote:
>>> Hi Prasanta,
>>>
>>> The fix looks ok, but for disposing the frame, instead of repeating 
>>> the same line twice, I would suggest to embed it into try ... finally 
>>> clause.
>>> Also, since "doTest" can technically throw exceptions, all those 
>>> calls should be included in the try block as well.
>>>
>>> Thanks,
>>> Krishna
>>>
>>> -----Original Message-----
>>> From: Prasanta Sadhukhan
>>> Sent: Tuesday, July 24, 2018 3:44 PM
>>> To: swing-dev at openjdk.java.net
>>> Subject: <Swing Dev> [12] RFR 
>>> JDK-8203904:javax/swing/JSplitPane/4816114/bug4816114.java: The 
>>> divider location is wrong
>>>
>>> Hi All,
>>>
>>> Please review a fix for an test problem where it is seen that the 
>>> test passes on windows 7 but fails on windows 10 citing the 
>>> JSplitPane divider location is wrong.
>>> This is because in windows 10, the frame border width/height is 
>>> different compared to windows 7 so splitpane is moved accordingly in 
>>> the frame.
>>>
>>> Proposed fix is to undecorate the frame so that border is not 
>>> considered and splitpane position is considered from top/left of the 
>>> frame itself.
>>> webrev: http://cr.openjdk.java.net/~psadhukhan/8203904/webrev.0/
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8203904
>>>
>>> Additionally, the test frame is also disposed which was not done 
>>> earlier.
>>> The test now passes in windows 7, windows 10, ubuntu 18, mac 10.13.
>>>
>>> Regards
>>> Prasanta
>>>
>>>
> 


-- 
Best regards, Sergey.



More information about the swing-dev mailing list