Integrated: 8288480: IGV: toolbar action is not applied to the focused graph

Roberto Castañeda Lozano rcastanedalo at openjdk.org
Mon Jun 20 07:38:57 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:
> 
> ![multi-view](https://urldefense.com/v3/__https://user-images.githubusercontent.com/8792647/173841115-084c6396-3843-4d9b-9951-f93c932100c3.png__;!!ACWV5N9M2RV99hQ!OI7MWyhgPcxaofvuVpr2naXNNL3ZQX9HOoggg8sU36CrGTDyiffsKqU0ihtxbEemR8nXMH-BHz0m5wtdgCflI3jc53wNVcHEXt4FzQ$ )
> 
> Tested manually by triggering the above actions within multiple split graph windows and asserting that they are only applied to their corresponding graphs.

This pull request has now been integrated.

Changeset: f62b2bd9
Author:    Roberto Castañeda Lozano <rcastanedalo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/f62b2bd9cda952b205ee03151cc58c95f588a742
Stats:     8 lines in 1 file changed: 0 ins; 7 del; 1 mod

8288480: IGV: toolbar action is not applied to the focused graph

Reviewed-by: chagedorn, thartmann

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

PR: https://git.openjdk.org/jdk/pull/9169


More information about the hotspot-compiler-dev mailing list