RFR: JDK-8269921 Text in Textflow and listeners on bounds can cause endless loop/crash and other performance issues [v4]

[Florian Kirmaier]

> It's "a bit" complicated.
> In some situations, getRuns get's called because listeners on bounds are set.
> This causes TextFlow to layout to compute the runs.
> Afterward, the bounds of the parents get updated. 
> This triggers a call to compute bounds - which cascades up to the children.
> When the geometry of the previous Text gets computed in this big stack - it throws an nullpointer.
> The Text doesn't have its runs, and calling TextFlow.layout is now a noop (it detects repeated calls in the same stack)
> 
> In the case it happened - it didn't repair and the application kinda crashed.
> This bug most likely can also be triggered by ScenicView or similar tools, which sets listeners to the bounds.
> It also can cause unpredictable performance issues.
> 
> Unit test and example stacktrace are in the ticket.
> 
> The suggested fix makes sure that recomputing the geometry of the Text, doesn't trigger the layout of the TextFlow. 
> The Textflow should be layouting by the Parent.
> This might change the behavior in some cases, but as far as I've tested it works without issues in TextFlow Heavy applications.
> 
> Benefits:
>  * Better Tooling Support For ScenicView etc.
>  * Fixes complicated but reproducible crashes
>  * Might fix some rare crashes, which are hard to reproduce
>  * Likely improves performance - might fix some edge cases with unpredictable bad performance

Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision:

  JDK-8269921
  Reverted the change to the layout, so we can fix the main-bug without further discussions.

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/564/files
  - new: https://git.openjdk.org/jfx/pull/564/files/a9516f3d..404f7103

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=564&range=03
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=564&range=02-03

  Stats: 4 lines in 1 file changed: 3 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jfx/pull/564.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/564/head:pull/564

PR: https://git.openjdk.org/jfx/pull/564