RFR: 8345039: IGV: save user-defined node colors to XML [v2]

Roberto Castañeda Lozano rcastanedalo at openjdk.org
Fri Nov 29 10:05:38 UTC 2024


On Fri, 29 Nov 2024 09:59:13 GMT, Tobias Holenstein <tholenstein at openjdk.org> wrote:

>> This PR depends on https://github.com/openjdk/jdk/pull/22402 
>> 
>> [JDK-8343535](https://bugs.openjdk.org/browse/JDK-8343535) introduced the possibility to give user-defined colors to a node.
>> The colors are lost when the user goes to the next graph or when IGV is closed. Save the colors as a property of the graph to the XML to make it more permanent. This requires that the user can also remove the colors again.
>> 
>> <img width="474" alt="color" src="https://github.com/user-attachments/assets/c994f93c-7db1-40e6-9963-2f5417cec77f">
>> 
>> ### Whats new
>> - Now colors are saved with the XML as well
>> - Colors are kept when changing to a different graph
>> - The user can remove the color again: This uses the color from the filter or WHITE otherwise
>
> Tobias Holenstein has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - patch2: fix botton on linux
>  - patch1

Looks good, making custom colors persistent really improves the usability of this feature!

-------------

Marked as reviewed by rcastanedalo (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22440#pullrequestreview-2469549831


More information about the hotspot-compiler-dev mailing list