<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">No issues in the awt tests, but I decide to update the fix in place.<div class=""><a href="http://cr.openjdk.java.net/~serb/8160270/webrev.00/src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java.sdiff.html" class="">http://cr.openjdk.java.net/~serb/8160270/webrev.00/src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java.sdiff.html</a></div><div class="">One line was added to check that we access the native peers only for HW components.<br class=""><div class=""><pre class=""><span class="new"> 657     GraphicsConfiguration getCurrentGraphicsConfiguration() {</span>
<span class="new">+658         Component hw = SunToolkit.getHeavyweightComponent(target);</span>
<span class="new"> 659         XWindow peer = AWTAccessor.getComponentAccessor().getPeer(hw);</span>
<span class="new"> 660         if (peer != null && peer.graphicsConfig != null) {</span>
<span class="new"> 661             return peer.graphicsConfig;</span>
<span class="new"> 662         }</span>
<span class="new"> 663         return graphicsConfig;</span>
<span class="new"> 664     }</span></pre><div class="">If there are no objections I’ll push this version.</div><div class=""><br class=""></div><div class="">:<br class=""><div><blockquote type="cite" class=""><br class="Apple-interchange-newline"><div class=""><div class="">It all looks plausible .. so +1 assuming no problems with the tests.<br class=""><br class="">-phil.<br class=""><br class="">On 03/16/2017 12:20 PM, Alexandr Scherbatiy wrote:<br class=""><blockquote type="cite" class=""><br class="">The fix looks good to me.<br class=""><br class="">Thanks,<br class="">Alexandr.<br class=""><br class="">On 3/16/2017 7:44 PM, Sergey Bylokhov wrote:<br class=""><blockquote type="cite" class="">Hello,<br class="">Please review the fix for jdk9.<br class=""><br class="">This bug was caused by the change in JDK8137571 where we started to use graphic configuration for each component to calculate the preferred size on the screen.<br class="">The problem is that in the code we had a few assumptions that the left-top corner of the screen is 0.0. Moreover for the Menu we do not update the GC when the window is moved to another screen(I’ll file a separate bug for this).<br class=""><br class="">Fix description:<br class="">- Menu components now tries to take the graphics configuration from the frame.<br class="">- All calculations are updated to take into account that the left-top corner is screenBounds.x/y.<br class="">- The choice component now updates the GC of its popup when its own gc is updated.<br class="">- I intentionally do not reformat the change to simplify review.<br class=""><br class="">I have started the jck/jtreg tests, they are still in progress I'll notify if some new issues will be found.<br class=""><br class="">Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8160270" class="">https://bugs.openjdk.java.net/browse/JDK-8160270</a><br class="">Webrev can be found at: <a href="http://cr.openjdk.java.net/~serb/8160270/webrev.00" class="">http://cr.openjdk.java.net/~serb/8160270/webrev.00</a><br class=""><br class=""></blockquote><br class=""></blockquote><br class=""></div></div></blockquote></div><br class=""></div></div></div></body></html>