<Swing Dev> RFR: JDK-8234386: [macos] NPE was thrown at expanding Choice from maximized frame

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Thu Nov 21 08:42:39 UTC 2019


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.


More information about the swing-dev mailing list