Integrated: 8088998: XYChart: duplicate child added exception when remove & add a series in several charts

[Karthik P K]

On Mon, 30 Jan 2023 12:21:38 GMT, Karthik P K <kpk at openjdk.org> wrote:

> While checking for duplicate series addition to the line chart, `setToRemove` value was not considered before throwing exception. Hence code to handling the case of adding the removed series was never run.
> 
> Added condition to check `setToRemove` boolean value before throwing exception. Made changes to call `setChart` method after calling `seriesBeingRemovedIsAdded`. Otherwise chart will not be drawn for the series, only points will be plotted.
> 
> This issue is reproducible only when animation is enabled because timeline will be still running when removed series is added back to the same chart. Since timeline does not run in unit tests, added system test to validate the fix.

This pull request has now been integrated.

Changeset: de8339d4
Author:    Karthik P K <kpk at openjdk.org>
Committer: Ajit Ghaisas <aghaisas at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/de8339d45a9e070598fa0ddc91c36fe74c67a956
Stats:     400 lines in 8 files changed: 378 ins; 3 del; 19 mod

8088998: XYChart: duplicate child added exception when remove & add a series in several charts

Reviewed-by: aghaisas, angorya

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

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