JMC-6241: Leaking Context Menu items

[Marcus Hirt]

Looks ok for 7.0.0!

Kind regards,
Marcus

On 2018-12-12, 22:24, "jmc-dev on behalf of Ken Dobson" <jmc-dev-bounces at openjdk.java.net on behalf of kdobson at redhat.com> wrote:

    Hi all,
    
    Josh has reviewed this however this still requires one more review. If
    someone has the time to give it a quick review that'd be great.
    
    Thanks,
    
    Ken Dobson
    
    On Thu, Dec 6, 2018 at 11:43 AM Joshua Matsuoka <jmatsuok at redhat.com> wrote:
    
    > Hi Ken,
    >
    > I applied the patch and it works well.
    >
    > Looks good.
    >
    > Cheers,
    >
    > - Josh
    >
    > On Tue, Dec 4, 2018 at 3:45 PM Ken Dobson <kdobson at redhat.com> wrote:
    >
    >> Hi all,
    >>
    >> This patch does a couple things.
    >> 1. It fixes the leaking context menu items which resulted in the context
    >> menu having multiple sets of the same thread activity lanes listed.
    >> 2. It adds a context menu to the Garbage Collections Page legend as there
    >> already was one for the Java Application page.
    >> 3. Fixes a small bug that lead to the Thread Activity lane on the GC page
    >> only refreshing if it was turned off and then back on again.
    >>
    >> If someone could give it a review that would be great.
    >>
    >> https://bugs.openjdk.java.net/browse/JMC-6241
    >>
    >> Thanks,
    >>
    >> Ken Dobson
    >>
    >>