<AWT Dev> [10] Review request for 8185634, 8185634: Java Fx-Swing dialogs appearing behind main stage

Semyon Sadetsky semyon.sadetsky at oracle.com
Mon Sep 25 15:36:47 UTC 2017


Hi Alexander,

On the Windows platform you've only modified the dialog native peer 
while on other platforms all window types are modified to use FX window 
as a parent. Shouldn't the frame native peer  be modified too on the 
Windows platform?

--Semyon


On 09/21/2017 10:56 PM, Alexander Zvegintsev wrote:
>
> Hi Phil,
>
> Please review the updated fix with reflection incorporated
> http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/
>
> New issue created JDK-8187803 
> <https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart 
> of this issue.
>
> Thanks,
> Alexander.
> On 21/09/2017 22:25, Phil Race wrote:
>> Some procedural comments :
>> Since 90% of this is in AWT code, I'd have thought awt-dev should be 
>> included here.
>> I've added that list.
>>
>> And apart from needing separate bug ids, I don't see why the bug 
>> below is confidential.
>>
>>
>> I agree with what Kevin pointed out off-line that as in the dialog 
>> case, the FX side
>> of the code can use reflection and simply be a harmless 
>> non-functional no-op
>> if the SwingAccessor does not provide the new method.
>>
>> BTW
>> 264 inline HWND GetOverridenHWnd() { return m_overridenHwnd; }
>> should be "dd" not "d".
>>
>> -phil.
>>
>> On 09/21/2017 03:38 AM, Alexander Zvegintsev wrote:
>>> Hello,
>>>
>>> please review the fix
>>>
>>> http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/
>>>
>>> for the issue
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8185634
>>>
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20170925/dd5c39f0/attachment.html>


More information about the awt-dev mailing list