Re: [10] Review request for 8155197: Focus transition issue

2017-10-04 Thread Sergey Bylokhov
On 10/4/17 03:25, Dmitry Markov wrote: Good catch! Actually we check restoreFocusTo before its usage, (i.e. we compare restoreFocusTo with most resent focus owner for the window; most recent focus owner is visible and focusable, otherwise it is removed from recent focus owners map). So the che

Re: [10] Review request for 8155197: Focus transition issue

2017-10-04 Thread Semyon Sadetsky
Hi Dmitry, If you make a detailed look on the Component class methods like setEnables(), setFocusable() and setVisible() the most recent focus owner is cleared or transferred only after the component is made non-focusable. Since the fix is aimed for concurrent scenarios the recheck actually m

Re: [10] JDK-8178361: JFileChooser does not allow to open folders with a double tap when using the touch screen interface

2017-10-04 Thread Anton Litvinov
Hello Shashi, Thank you for understanding and confirmation that my alternative fix for this bug can be reviewed further. I have taken ownership over this bug. Thank you, Anton On 04/10/2017 06:08, Shashidhara Veerabhadraiah wrote: Hi Anton, Thanks for this information and please go ahead wi

Re: [10] Review request for 8155197: Focus transition issue

2017-10-04 Thread Dmitry Markov
Hi Sergey, Good catch! Actually we check restoreFocusTo before its usage, (i.e. we compare restoreFocusTo with most resent focus owner for the window; most recent focus owner is visible and focusable, otherwise it is removed from recent focus owners map). So the checks before assignment are not