RFR: 8262725: IGV: crash when removing all graphs in a group [v2]
rrich at openjdk.java.net
Mon Apr 19 13:56:39 UTC 2021
On Mon, 19 Apr 2021 13:28:20 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:
>> This change makes IGV close graph views when their groups are either a) removed or b) emptied (all graphs are removed but the group remains), and avoids faulty graph view computation in case b).
>> Tested the following scenarios manually (on groups loaded both via network and from a file, and on JDK 8, 11, and 15):
>> 1. open a graph, then remove all graphs in the group (as described in the [bug report](https://bugs.openjdk.java.net/browse/JDK-8262725));
>> 2. open a graph, then remove its group; and
>> 3. open a graph, then remove all graphs and groups (File -> Remove all graphs and groups).
> Roberto Castañeda Lozano has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> - Simplify listener registration
> - Merge branch 'master' into JDK-8262725
> - Close graph views when their groups are removed or emptied
> The preceding fix (https://github.com/openjdk/jdk/pull/3515) should make this possible, however I see now that it wasn't merged into this PR. I merged it now ([04c0b3c](https://github.com/openjdk/jdk/commit/04c0b3c8ec9896a8e26d72af3461069c5cc5ae91)), could you please try again?
Working nicely now!
More information about the hotspot-compiler-dev