Re: [9] Review request for 8178996 [macos] JComboBox doesn't display popup in mixed JavaFX Swing Application on 8u131 and Mac OS 10.12

2017-05-11 Thread Sergey Bylokhov
Ok, then looks fine. - alexander.zvegint...@oracle.com wrote: > I thought about that, by I decided to add this flag for code > readability. > > -- > Thanks, > Alexander. > > On 11.05.2017 18:34, Sergey Bylokhov wrote: > > Hi, Alex. > > Why you need a separate flag since you can use the

Re: [9] Review request for 8178996 [macos] JComboBox doesn't display popup in mixed JavaFX Swing Application on 8u131 and Mac OS 10.12

2017-05-11 Thread Semyon Sadetsky
Looks good. --Semyon On 05/11/2017 01:54 AM, Alexander Zvegintsev wrote: Hi all, please review the fix http://cr.openjdk.java.net/~azvegint/jdk/9/8178996/00/ for the issue https://bugs.openjdk.java.net/browse/JDK-8178996 Popup doesn't show up due to owner is lightweight frame and

[9] Review request for 8178996 [macos] JComboBox doesn't display popup in mixed JavaFX Swing Application on 8u131 and Mac OS 10.12

2017-05-11 Thread Alexander Zvegintsev
Hi all, please review the fix http://cr.openjdk.java.net/~azvegint/jdk/9/8178996/00/ for the issue https://bugs.openjdk.java.net/browse/JDK-8178996 Popup doesn't show up due to owner is lightweight frame and doesn't have a native pointer, thus we do not invoke owner.execute() code block,