<AWT Dev>  Review request for 8169589: [macosx] Activating a JDialog puts to back another dialog
dmitry.markov at oracle.com
Sat Nov 26 13:01:17 UTC 2016
I have added some remarks to the code. The updated webrev is located at
Proposed functionality performs ordering operation from the very bottom,
(i.e. root owner) so that the windows are ordered above their nearest
parent; ancestors of the window, which is going to become ‘main window’,
are placed above their siblings.
Summary of changes:
- orderAboveSiblings() is responsible for retrieval of root owner and
initial creation of the list of the windows which have to be ordered.
- orderAboveSiblingsImpl(Window windows) performs ordering of the
windows specified by input array. If the window is one of ancestors of
'main window' or is going to become main by itself, the window will be
ordered above its siblings; otherwise the window is just ordered above
its nearest parent. This method is recursively called until all windows
in window hierarchy are ordered.
- Two helper methods: getRootOwner() is responsible for retrieval of
root owner for the window and isOneOfOwnersOrSelf(CPlatformWindow
window) - tests whether the current window is one of ancestors of the
> On 25 Nov 2016, at 16:16, Sergey Bylokhov <Sergey.Bylokhov at oracle.com>
> 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:
>> 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.
>> 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.
> Best regards, Sergey.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the awt-dev