Hi Sergey, Please find the updated webrev. It is true that the
notifications are unique to each window and it should be a instance
variable.
http://cr.openjdk.java.net/~sveerabhadra/8213071/webrev.01/
I have reproduced this issue on Prashanth's mac as well with JDK 8
ver:121. Please note the behavior is different with respect to the voice
over state is on or off.
Thanks and regards,
Shashi
On 22/11/18 6:03 AM, Sergey Bylokhov wrote:
On 21/11/2018 16:00, Sergey Bylokhov wrote:
Can you please double check it on a different systems,
I tried to reproduce it a few time on the current client and it seems
to work.
And the question about the current fix, are you sure that the static
boolean flag will
work properly if the app will have a few windows? I guess it should be
perWindow flag?
On 12/11/2018 22:28, shashidhara.veerabhadra...@oracle.com wrote:
Hi Sergey, This same behavior exists on jdk 7 and jdk 8 as well. So
it is not a regression I think.
Thanks and regards,
Shashi
On 13/11/18 6:12 AM, Sergey Bylokhov wrote:
Hi, Shashi.
Did you check this behavior was in jdk7/8 GA, or it is a regression?
On 11/11/2018 04:58, shashidhara.veerabhadra...@oracle.com wrote:
Hi All, Please review a fix for an accessibility vpat issue.
Bug: https://bugs.openjdk.java.net/browse/JDK-8213071
Webrev: http://cr.openjdk.java.net/~sveerabhadra/8213071/webrev.00/
The behavior of the same actions to reproduce the problem,
produced different results with VO and without VO. It have behaved
right without the VO and with VO, the focus was lost.
The issue is with window being the KEY and MAIN window per the Mac
windowing system. There are platform notifications like the
'windowDidBecomeMain' and 'windowDidBecomeKey' are issued for the
window that is about to get the focus and key strokes. So
typically under normal conditions, the window which is about to
get notified of becoming the focusable window would get the KEY
and MAIN notifications in that order.
But with VO, some how the KEY notification was not delivered to
java window and it only receives the MAIN notification. In this
scenario, though the isKeyWindow() is true for this window, KEY
notification was not delivered and only the MAIN notification is
delivered. We do transfer the focus under the KEY notification
handler which was not getting executed since it received no KEY
notification. Per me, isKeyWindow() and MAIN notification is
sufficient to transfer the focus and become a main window but I
have not disturbed the existing handlers for the notification and
fixed only for the case which is causing this issue.
Thanks and regards,
Shashi
