[8u-dev] Request for Approval: 8158325: Memory leak in com.apple.laf.ScreenMenu: removed JMenuItems are still referenced

[Rob McKenna]

Apologies Robin, this one flew under the radar.

Approved.

You will need a sponsor to push the change for you if you are not a jdk8u committer. The best place to start would be the review thread. Alexandr may be willing to help.

	-Rob

On 06/07/16 05:48, Robin Stevens wrote:
> Hello,
> 
> do I need to perform any additional steps to get approval for this backport
> ? If so, can somebody let me know which steps.
> If I get approval, can somebody also tell me what I still need to do to do
> the actual backport ?
> 
> Thanks,
> 
> Robin
> 
> On Thu, Jun 30, 2016 at 11:27 PM, Robin Stevens <stevensro at gmail.com> wrote:
> 
> > Please approve backport of 8158325 to 8u-dev.
> >
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8158325
> > Webrev:
> > http://cr.openjdk.java.net/~alexsch/robin.stevens/8158325/webrev.01
> > Review thread:
> > http://mail.openjdk.java.net/pipermail/swing-dev/2016-June/006166.html
> >
> > Changes apply cleanly to jdk8u-dev after path reshuffling. Note that I
> > needed to add:
> >
> > jdk/src/java.desktop/macosx/classes/com/apple/laf :
> > jdk/src/macosx/classes/com/apple/laf
> >
> > to common/bin/unshuffle_list.txt before the unshuffle script could shuffle
> > the changes.
> >
> > Thanks,
> >
> >
> > Robin
> >