Hi, Toshio. In a few lines above your fix, the code takes care of null GC and use the main screen size as a screen bound, so I think your change should calculate Insets for the main screen as well instead of using zeros.
BTW probably GraphicsEnvironment.getMaximumWindowBounds() could be used at line 336 additionally to your fix. On 11/19/19 7:55 pm, Toshio 5 Nakamura wrote:
Hi All, Could you review the following fix? Also, I'd like to ask a sponsor of this fix, since I'm not a committer. Bug: https://bugs.openjdk.java.net/browse/JDK-8234386 Webrev: http://cr.openjdk.java.net/~tnakamura/8234386/webrev.00 Issue: NullPointerException was thrown when Choice was expanded from maximized Frame. Fix: Simply adding a null check to JPopupMenu. When the frame was maximized and expanding Choice component, GraphicsConfiguration parameter can be null. Detailed description was in JBS. Thanks, Toshio Nakamura
-- Best regards, Sergey.
