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

[Robin Stevens]

Thanks Rob for the approval.

I send a new message in the original review thread and hope someone will
pick it up and commit the changes.

Robin

On Thu, Jul 7, 2016 at 2:41 PM, Rob McKenna <rob.mckenna at oracle.com> wrote:

> 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
> > >
>