> Fixing a memory leak. > A node hard references its old parent after CSS layout and getting removed. > This shouldn't be the case, this is very counterintuitive. > > The fix uses a WeakReference in CSSStyleHelper for firstStyleableAncestor. > This should be fine because the CSS should only depend on it if it's still > the real parent. > In that case, it doesn't get collected.
Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: 8263402 Rewrote the style memoryleak test ------------- Changes: - all: https://git.openjdk.java.net/jfx/pull/424/files - new: https://git.openjdk.java.net/jfx/pull/424/files/b39db419..52fa05f6 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jfx&pr=424&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=424&range=01-02 Stats: 61 lines in 1 file changed: 31 ins; 27 del; 3 mod Patch: https://git.openjdk.java.net/jfx/pull/424.diff Fetch: git fetch https://git.openjdk.java.net/jfx pull/424/head:pull/424 PR: https://git.openjdk.java.net/jfx/pull/424