RFR: 7120: Charts in custom pages are not auto refreshed
Alex Macdonald
aptmac at openjdk.java.net
Thu Feb 4 22:33:46 UTC 2021
This PR addresses JMC-7120 [0], in which charts in the custom pages are not auto refreshed. This issue is a regression from the changes to the `ChartCanvas` in the threads page enhancement, sorry about that.
The culprit here was the wrongful removal of a `redrawChart()` call in `ChartCanvas.setChart`. Looking back through the old commits I have for the Threads Page work I couldn't find where it was removed, but it must have been pretty early into the work. I noticed that the `redrawChart()` is present in JMC 7 [1] and in the commit prior to the threads page pr going in [2], but not afterwards [3]. This PR simply adds back the redraw call.
gif:

[0] https://bugs.openjdk.java.net/browse/JMC-7120
[1] https://hg.openjdk.java.net/jmc/jmc7/file/63ec7d0ee8d9/application/org.openjdk.jmc.ui/src/main/java/org/openjdk/jmc/ui/misc/ChartCanvas.java#l533
[2] https://github.com/openjdk/jmc/blob/c3f49204ce36139ac83ccfa88b2ada26a0b2cd04/application/org.openjdk.jmc.ui/src/main/java/org/openjdk/jmc/ui/misc/ChartCanvas.java#L533
[3] https://github.com/openjdk/jmc/blob/5feaf25fa2a4a61236a733053dc60134c89efccf/application/org.openjdk.jmc.ui/src/main/java/org/openjdk/jmc/ui/misc/ChartCanvas.java#L716
-------------
Commit messages:
- 7120: Charts in custom pages are not auto refreshed
Changes: https://git.openjdk.java.net/jmc/pull/211/files
Webrev: https://webrevs.openjdk.java.net/?repo=jmc&pr=211&range=00
Issue: https://bugs.openjdk.java.net/browse/JMC-7120
Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jmc/pull/211.diff
Fetch: git fetch https://git.openjdk.java.net/jmc pull/211/head:pull/211
PR: https://git.openjdk.java.net/jmc/pull/211
More information about the jmc-dev
mailing list