<Swing Dev> Review request for 8016665: [macosx] JComponent behaviour doesn't comply API documentation (setComponentOrientation method), Aqua LAF

Avik Niyogi avik.niyogi at oracle.com
Wed Jan 13 06:27:20 UTC 2016


Hi Sergey,
If fFileList refers to a soft linked empty folder, we will  will not need to apply orientation to it’s sub-components.

With Regards,
Avik Niyogi

> On 12-Jan-2016, at 7:19 pm, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
> 
> Hi, Avik.
> Is it possible that fFileList can be null? I see that in other places we do not check it to null?
> 
> On 04/01/16 11:52, Avik Niyogi wrote:
>> Hi All,
>> 
>> Please review the webrev.01 :
>> http://cr.openjdk.java.net/~aniyogi/8016665/webrev.01/ <http://cr.openjdk.java.net/~aniyogi/8016665/webrev.01/>
>> incorporated with the inputs received.
>> 
>> With Regards,
>> Avik Niyogi
>> 
>>> On 28-Dec-2015, at 10:23 am, Avik Niyogi <avik.niyogi at oracle.com <mailto:avik.niyogi at oracle.com>
>>> <mailto:avik.niyogi at oracle.com <mailto:avik.niyogi at oracle.com>>> wrote:
>>> 
>>> Hi Alexandr,
>>> 
>>> Automated test may fail based on folder contents on individual systems
>>> irrespective of the fix directly not depending on the same.
>>> Also, to confirm this fix, it will need visual confirmation and hence,
>>> no automated test was provided.
>>> 
>>> With Regards,
>>> Avik Niyogi
>>>> On 23-Dec-2015, at 7:32 pm, Alexander Scherbatiy
>>>> <alexandr.scherbatiy at oracle.com <mailto:alexandr.scherbatiy at oracle.com>
>>>> <mailto:alexandr.scherbatiy at oracle.com <mailto:alexandr.scherbatiy at oracle.com>>> wrote:
>>>> 
>>>> 
>>>> 
>>>>  The fix looks good to me.
>>>> 
>>>>  Is it possible to write an automated test for the fix?
>>>> 
>>>>  Thanks,
>>>>  Alexandr.
>>>> 
>>>> On 12/21/2015 2:55 PM, Avik Niyogi wrote:
>>>>> Hi All,
>>>>> 
>>>>> Kindly review the bug fix for JDK 9.
>>>>> 
>>>>> *Bug:*
>>>>> https://bugs.openjdk.java.net/browse/JDK-8016665 <https://bugs.openjdk.java.net/browse/JDK-8016665>
>>>>> 
>>>>> *Webrev:*
>>>>> http://cr.openjdk.java.net/~aniyogi/8016665/webrev.00/ <http://cr.openjdk.java.net/~aniyogi/8016665/webrev.00/>
>>>>> <http://cr.openjdk.java.net/%7Eaniyogi/8016665/webrev.00/ <http://cr.openjdk.java.net/%7Eaniyogi/8016665/webrev.00/>>
>>>>> 
>>>>> *Issue:*
>>>>> The manual test: Swing_AllComponents/Manual/I18nSwingTests
>>>>> in testsuite fails.
>>>>> 
>>>>> *Cause:*
>>>>> Due to not honouring of RIGHT_TO_LEFT parameter for setOrientation
>>>>> method applied for a JFileChooser for the AquaLookAndFeel only,
>>>>> the fileChooser does not get displayed in RIGHT_TO_LEFT orientation.
>>>>> This issue was verified to exist only in AquaLookAndFeel for
>>>>> JFileChooser only due to wrong implementation in AquaFileSystemModel.
>>>>> Also, as provided in comments: "The Aqua LAF must support the RTL
>>>>> orientation of JFileChooser."
>>>>> 
>>>>> *Fix:*
>>>>> Added implementation for the check of RIGHT_TO_LEFT
>>>>> ComponentOrientation and verified with test suite.
>>>>> 
>>>>> 
>>>>> With Regards,
>>>>> Avik Niyogi
>>>> 
>>> 
>> 
> 
> 
> -- 
> Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20160113/32db953e/attachment.html>


More information about the swing-dev mailing list