RFR: 8252935: Add treeShowing listener only when needed [v4]

John Hendrikx jhendrikx at openjdk.java.net
Mon Feb 1 04:44:03 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 four additional commits since the last revision:

 - Add missing TreeShowingExpressionTest which also tests SubScene
 - Also dispose listeners on Window/Showing in dispose
 - Fix review comments
 - Update copyrights

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

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/185/files
  - new: https://git.openjdk.java.net/jfx/pull/185/files/e68a886d..2b6df779

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jfx&pr=185&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=185&range=02-03

  Stats: 342 lines in 7 files changed: 325 ins; 0 del; 17 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