Hello, AWT team. This is a reminder.
With best regards. Petr. On 11 июля 2014 г., at 1:59, Anthony Petrov <[email protected]> wrote: > Hi Petr, > > I'm fine with the targeted fix. We often do a similar thing in JavaFX when > processing various events, so the approach is proven to work good. > > However, generally I agree with your comment from the bug report about the > necessity to process dispose selectors in the outer event loop only. IIRC, we > do something similar in the WToolkit native implementation. So I suggest to > file a separate medium priority bug to investigate this further for JDK 9 (or > tbd-major). > > -- > best regards, > Anthony > > On 7/10/2014 5:02 PM, Petr Pchelko wrote: >> Hello, AWT team. >> >> Please review a small fix for the issue: >> https://bugs.openjdk.java.net/browse/JDK-8032864 >> The fix is available at: >> http://cr.openjdk.java.net/~pchelko/9/8032864/webrev.01/ >> >> I failed to create a general fix for the core problem, so I've made a point >> fix for this bug. The problem is that CAccessible.getFocusOwner opens a >> nested loop and goes to EDT. >> This leads to some unknown reordering of selectors, and we crash. >> retain/releasing the window around the call to getFocusOwner fixes this >> particular bug. >> I've trued to make a minimal fix as it needs to be backported to 8 and 7. >> I've checked for memory leaks using Instruments. >> >> With best regards. Petr. >>
