Re: [8] [PATCH] Review request for 8001633: Wrong alt processing during switching between windows.

[Anton V. Tarasov]

Looks ok to me.

Thanks,
Anton.

On 11/1/12 9:32 PM, Mikhail Cherkasov wrote:

Could you please review this fix again?
There's no changes in fix, but test was added.
Webrev:

http://cr.openjdk.java.net/~alexp/8001633/webrev.02/ <http://cr.openjdk.java.net/%7Ealexp/8001633/webrev.02/>

On 10/29/2012 2:23 PM, Mikhail Cherkasov wrote:

Hi all,

Could you please review a fix for 8001633 <http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8001633>: Wrong alt processing during switching between windows.

Bug:http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8001633
Webrev:
http://cr.openjdk.java.net/~bagiras/8/7082294.1/

To prevent wrong focus traversing to menu was added additional check to AltProcessor. It checks that original source of 'alt' event belongs to AltProcessor.winAncestor

or its component.

Patch is attached.

Thanks,

Mikhail.