<AWT Dev>  [PATCH] Review request for 8001633: Wrong alt processing during switching between windows.
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);
>> 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>
>>> 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.
More information about the awt-dev