[Rev 03] RFR: 8236259: MemoryLeak in ProgressIndicator

[Kevin Rushforth]

On Tue, 10 Mar 2020 14:34:57 GMT, Florian Kirmaier <fkirmaier at openjdk.org> wrote:

>> The fix looks good to me. I've verified that it fixes the leak.
>> 
>> The newly added test looks good as well, with some comments left inline (formatting, a missing copyright header, and a
>> couple other suggestions).
>
> @kevinrushforth
> I've now cleaned it up based on your suggestions.
> It would be great when Travis could validate the formatting.
> It would save a lot of time for everyone.

Thanks, I'll take a look.

As for automating this, we no longer have Travis builds hooked up. I have a pending task to look into using GitHub
Actions to do a sanity build / test (which would be a big help). We could look into adding a checkstyle task to that to
check for at least some of the style guidelines.

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

PR: https://git.openjdk.java.net/jfx/pull/71