RFR: 8081474: SwingWorker calls 'done' before the 'doInBackground' is finished [v10]

[Alexey Ivanov]

On Thu, 2 Feb 2023 21:28:18 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> But it did not work as expected before the fix as well since the "notification and done" work in parallel anyway.
>
> It could be tested by flushing and blocking the EDT in the listener. That will allow us to check the order of events posted from the listener and from the "doneEDT".

I haven't verified it, but I think it worked as it's stated: `done` is called first, then listeners are notified. According to the order of calls

https://github.com/openjdk/jdk/blob/b00b70c2400d28070d26630614a010bc52237827/src/java.desktop/share/classes/javax/swing/SwingWorker.java#L310-L313

`SwingWorker.done` gets invoked first followed by `setState` which, in its turn, notifies the listeners.

At this time, as you pointed out, the order is reversed:

https://github.com/openjdk/jdk/blob/76b02ea3a8add954705d6df7f91f5896d141b02d/src/java.desktop/share/classes/javax/swing/SwingWorker.java#L307-L308

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

PR: https://git.openjdk.org/jdk/pull/11940