Looks fine.
On 9/23/17 08:25, Dmitry Markov wrote:
Hello,
Could you review a fix for jdk10, please?
bug: https://bugs.openjdk.java.net/browse/JDK-8155197
webrev: http://cr.openjdk.java.net/~dmarkov/8155197/webrev.00/
Problem description:
Currently we restore focus synchronously to a component if the window (which
contains it) owns the focus. However if another window/dialog is displayed at
the same time, the first window loses focus, (i.e. synchronous focus request
failed) and the focus target is shifted to the next component in the focus
traversal cycle.
Fix:
It is necessary to check the result of synchronous focus request for a
component. If the focus request’s failed because the contained window lost
focus, we should restore focus to this component next time when its parent
window obtains focus.
Testing:
I ran corresponding JCK and regression tests and did not observe any new
failures.
Thanks,
Dmitry
--
Best regards, Sergey.
