JMC-6367 - Improve Thread graph tooltip

Langer, Christoph christoph.langer at
Fri Feb 15 15:28:33 UTC 2019

Hi Alex,

I had a look at your change. As I'm quite new to JMC it was merely to learn something ��

However, I could verify that your fix works and improves things in the thread lanes. I also couldn't find obvious mistakes.

But as newbie and since I'm no reviewer here I have to defer the real review to the JMC experts.

Best regards

> -----Original Message-----
> From: jmc-dev <jmc-dev-bounces at> On Behalf Of Alex
> Macdonald
> Sent: Donnerstag, 14. Februar 2019 22:22
> To: jmc-dev at
> Subject: RFR: JMC-6367 - Improve Thread graph tooltip
> Hi,
> The following webrev [0] addresses issue JMC-6367 [1], in which the thread
> lane tooltips should display descriptive information (thread & lane names)
> pertaining to the highlighted lane.
> The information to be displayed in the tooltip had to be carried in the
> Span object which is passed to the ChartToolTipProvider. There was an
> unused String description argument in the QuantitySpanRenderer which fits
> this need, and allows for the passing of the tooltip title from the
> ThreadGraphLanes through the QuantitySpanRenderer to the
> QuantitySpanRendering. This change was also mirrored in the SpanRenderer.
> If the Span obj has a description when it hits the ChartToolTipProvider, a
> title will be appended to the tooltip.
> I've created an imgur album [2] to show some examples. Some images
> illustrate the before and after of this patch when viewing tooltips for a
> thread [3][4], and viewing a thread that corresponds to an active filter
> (e.g., Java Latencies lane) [5][6].
> Thoughts?
> Cheers,
> Alex
> [0]
> [1]
> [2]
> [3]
> [4]
> [5]
> [6]

More information about the jmc-dev mailing list