<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