<AWT Dev> [8] [PATCH] Review request for 8001633: Wrong alt processing during switching between windows.

Leonid Romanov leonid.romanov at oracle.com
Mon Oct 29 12:14:05 PDT 2012


Looks OK to me.

On Oct 29, 2012, at 8:18 PM, Mikhail Cherkasov <mikhail.cherkasov at oracle.com> wrote:

> Please review the second version: http://cr.openjdk.java.net/~alexp/8001633/webrev.01/ <http://cr.openjdk.java.net/%7Ealexp/8001633/webrev.01/>
> All remarks was corrected.
> 
> On 10/29/2012 4:32 PM, Anton V. Tarasov wrote:
>> Hi Mikhail,
>> 
>> * KeyEvent.java
>> 
>> - No need to initialize 'originalSource' in constructors which call to this(...) where you already initialized it.
>> 
>> -950      * we need to able to obtain original source.
>> 
>> "be" is omitted ("we need to be able")
>> 
>> * WindowsRootPaneUI.java
>> 
>> I think there's no need to put another 'skip' setting into a separate if-block (skip will be equal 'false' in majority of cases).
>> Why don't you write it simply as follows?
>> 
>> Component originalSource = AWTAccessor.getKeyEventAccessor().getOriginalSource(ev);
>> 
>> skip = (ev.getWhen()<= ((SunToolkit)tk).getWindowDeactivationTime(winAncestor)) ||
>>     SwingUtilities.getWindowAncestor(originalSource) != winAncestor);
>> 
>> 
>> Thanks,
>> Anton.
>> 
>> On 29.10.2012 14:23, 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 <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.
>>> 
>> 
> 
> <altProcessing2.patch>




More information about the awt-dev mailing list