RFR: 8252935: Add treeShowing listener only when needed [v5]
John Hendrikx
jhendrikx at openjdk.java.net
Tue Feb 9 20:56:00 UTC 2021
> This is a PoC for performance testing.
>
> It contains commented out code in PopupWindow and ProgressIndicatorSkin and two tests are failing because of that.
>
> This code avoids registering two listeners (on Scene and on Window) for each and every Node to support the aforementioned classes. The complete change should update these two classes to add their own listeners instead.
John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
Apply changes from review comments
-------------
Changes:
- all: https://git.openjdk.java.net/jfx/pull/185/files
- new: https://git.openjdk.java.net/jfx/pull/185/files/2b6df779..02b4272e
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jfx&pr=185&range=04
- incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=185&range=03-04
Stats: 71 lines in 3 files changed: 31 ins; 32 del; 8 mod
Patch: https://git.openjdk.java.net/jfx/pull/185.diff
Fetch: git fetch https://git.openjdk.java.net/jfx pull/185/head:pull/185
PR: https://git.openjdk.java.net/jfx/pull/185
More information about the openjfx-dev
mailing list