> 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.
Karthik P K has updated the pull request incrementally with one additional commit since the last revision: Addressing review comments ------------- Changes: - all: https://git.openjdk.org/jfx/pull/1015/files - new: https://git.openjdk.org/jfx/pull/1015/files/0282896e..d707f321 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=1015&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1015&range=01-02 Stats: 54 lines in 7 files changed: 18 ins; 34 del; 2 mod Patch: https://git.openjdk.org/jfx/pull/1015.diff Fetch: git fetch https://git.openjdk.org/jfx pull/1015/head:pull/1015 PR: https://git.openjdk.org/jfx/pull/1015
