RFR: 8289357: (Tree)TableView is null in (Tree)TableRowSkin during autosize [v2]
Marius Hanl
mhanl at openjdk.org
Tue Jun 28 18:46:38 UTC 2022
> Initialize the `(Tree)TableView` when creating the measure row.
> This will guarantee, that we can access the `(Tree)TableView` in the `(Tree)TableRowSkin`, which is currently only null during the autosizing (It is always set otherwise).
>
> With this change, a NPE is happening as the `(Tree)TableRow` currently assumes, that there must be a `VirtualFlow` somewhere in the scene (parent). I guard against this now.
> I remembered, that there is a ticket for the above behaviour here: https://bugs.openjdk.org/browse/JDK-8274065
>
> Finally, the `(Tree)TableRow` must be removed after the autosizing and the index must be set to `-1` (as for the cell) so that e.g. `cancelEdit` is not triggered twice. Some tests catched that (see `test_rt_31015`).
Marius Hanl has updated the pull request incrementally with one additional commit since the last revision:
8289357: Remove (Tree)TableRow after autosizing and update the index to -1 to prevent triggering of listener
-------------
Changes:
- all: https://git.openjdk.org/jfx/pull/805/files
- new: https://git.openjdk.org/jfx/pull/805/files/1c23a030..e3a9545e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=805&range=01
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=805&range=00-01
Stats: 8 lines in 1 file changed: 4 ins; 2 del; 2 mod
Patch: https://git.openjdk.org/jfx/pull/805.diff
Fetch: git fetch https://git.openjdk.org/jfx pull/805/head:pull/805
PR: https://git.openjdk.org/jfx/pull/805
More information about the openjfx-dev
mailing list