RFR: 6531: Make the flame chart view render labels on windows

Marcus Hirt hirt at openjdk.java.net
Mon Jan 27 17:00:43 UTC 2020

On Mon, 27 Jan 2020 16:29:23 GMT, Alex Macdonald <aptmac at openjdk.org> wrote:

>> How hard would it be to make SWT use Edge as the default embedded browser and fall back to IE if not available?
>> It doesn't feel right to me to maintain d3-flame-graph fork, as @thegreystone stated it's a bit scary.
> As for "maintaining the fork", the next version up from the fork (v2.0.3) is incompatible with IE, so the fork is basically a one-shot fix that can't be updated. The interesting thing here is that we're using v2.0.3 (Sept 2018) , so if we had used just one release prior then the flame labels would have rendered in JMC Windows (without text) prior to the latest commits for colouring.
>> I'd propose the way @thegreystone has stated -> SWT update to use an alternative (newer) browser.
> Chromium support was initially slated for integration Q1 2017 [0], and the related patch looked to be on track to be posted by the end of 2019 [1], but it hasn't made it out yet. So when the patch does get posted, it'll need review, inclusion into SWT, and then shipped with a release .. we're looking at potentially quite a bit of time before this happens, but hopefully not.
> I'm conflicted both ways, so I'll go based on the consensus here. I'm not sure how popular JMC is for Windows, or how many people are wanting (or trying) to use the flame view, but it doesn't sit right with me that the view doesn't display anything. Maybe it can print an error screen mentioning the incompatibility instead?
> [0] https://projects.eclipse.org/development_effort/implement-swtchromium-integration
> [1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=549585#c20

I think I'm ok with this being a stop-gap measure. It does make third party-dependencies a bit funny - and there is a bit of trust involved in getting the javascript from a private fork of a project, and even a private push of the resulting build to a CDN. As soon as the Eclipse changes are in, in any form and shape, we should switch.


PR: https://git.openjdk.java.net/jmc/pull/41

More information about the jmc-dev mailing list