<AWT Dev> [9] Review request for 8169589: [macosx] Activating a JDialog puts to back another dialog

[Sergey Bylokhov]

Hi, Dmitry.
Can you please adds some comments to the code and describe what is going on.

On 25.11.16 16:08, dmitry markov wrote:
> Hello,
>
> Could you review a fix for jdk9, please?
>
>     bug: https://bugs.openjdk.java.net/browse/JDK-8169589
>     webrev: http://cr.openjdk.java.net/~dmarkov/8169589/webrev.00/
>
> Problem description:
> Current implementation of CPlatformWindow.orderAboveSiblings() just
> recursively pops up the windows from ‘active’ parent-child window chain.
> At the same time other child windows (which are not in active chain)
> stayed ‘untouched’ and may be placed behind their nearest parent/owner.
>
> Fix:
> CPlatformWindow.orderAboveSiblings() should be modified. It has to take
> into account that a window may own more than one child window.
>
> Note: JCK tests passed on the build with the fix.
>
> Thanks,
> Dmitry


-- 
Best regards, Sergey.