RFR: 8288480: IGV: toolbar action is not applied to the focused graph
Tobias Hartmann
thartmann at openjdk.org
Mon Jun 20 06:26:55 UTC 2022
On Wed, 15 Jun 2022 13:45:53 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:
> When multiple graphs are displayed simultaneously in split windows, the following toolbar actions are always applied to the same graph, regardless of which graph window is focused:
>
> - search nodes and blocks
> - extract node
> - hide node
> - show all nodes
> - zoom in
> - zoom out
>
> This changeset ensures that each of the above actions is only applied within its corresponding graph window. This is achieved by applying the actions to the graph window that is currently activated (`EditorTopComponent.getRegistry().getActivated()`) instead of the first matching occurrence in `WindowManager.getDefault().getModes()`.
>
> The changeset makes it practical, for example, to explore different views of the same graph simultaneously, as illustrated here:
>
> 
>
> Tested manually by triggering the above actions within multiple split graph windows and asserting that they are only applied to their corresponding graphs.
Looks good.
-------------
Marked as reviewed by thartmann (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9169
More information about the hotspot-compiler-dev
mailing list