Integrated: 8349091: Charts: exception initializing in a background thread

Andy Goryachev angorya at openjdk.org
Wed Mar 5 18:17:14 UTC 2025


On Thu, 6 Feb 2025 19:26:25 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

> Root Cause:
> (Multiple) properties are getting bound to the global `Platform.accessibilityActive` property. Binding (and I say, accessing) of properties is not thread-safe.
> 
> I also changed the design a bit.  Originally, every symbol in a chart had its `focusTraversableProperty` bound to `Platform.accessibilityActive` in order to enable the accessibility navigation across the chart data points.  This is rather inefficient, as the property has to manage (thousands?) of listeners.
> 
> Instead, a single boolean property is added to each chart, with a listener added to it which iterates over data symbols to toggle the `focusTraversableProperty` directly.
> 
> The exact moment when the new property gets bound is also important, and has to happen in the FX application thread.
> 
> With this change, it is safe to create and populate charts with data in a background thread.
> 
> ---
> 
> ## NOTES
> 
> 1. It looks like the `Platform.accessibilityActive` property never transitions back to false  after it transitioned to true.  Some say it is because there is no mechanism in the platform to get notified which cannot possibly be true.
> 2. The javadoc for `Platform.accessibilityActiveProperty()` method stipulates that "_This method may be called from any thread_" which is patently not true as the current implementation is simply not thread-safe.
> 
> ## Note to the Reviewers
> 
> To avoid merge conflicts, the preferred order of integrations:
> 
> #1697 
> #1713 
> #1717

This pull request has now been integrated.

Changeset: fc770fb9
Author:    Andy Goryachev <angorya at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/fc770fb96bdd6804c4b7633a62cbb2fe0a50345f
Stats:     316 lines in 11 files changed: 176 ins; 101 del; 39 mod

8349091: Charts: exception initializing in a background thread

Reviewed-by: kizune, kcr

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

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


More information about the openjfx-dev mailing list