Sorry to interrupt you but since I do know the test let me say that requestFocus() is an important part of the test, if you are going to replace it with requestFocusInWindow() you (effectively) remove it from list of regression tests. Please check 4369903 bug for more information.
Regards, Oleg. P.S. feel free to ask more questions if you need. On Fri, Nov 14, 2014 at 5:07 PM, Anton V. Tarasov <anton.tara...@oracle.com> wrote: > Hi Mikhail, > > Looks fine for me. Thanks! This was an old one... Do you have any plans to > fix it in 8/9 as well? > > Regards, > Anton. > > > On 14.11.2014 18:57, mikhail cherkasov wrote: >> >> Hello all, >> >> Could you please review a simple fix of closed test, the test was moved to >> open repo >> and requestFocus was replaced requestFocusInWindow. >> >> Because "requestFocus" cause infinite war for focus between two windows, >> however this >> behavior is correct and doesn't violet spec. >> >> [TESTBUG] closed/java/awt/Focus/ActivateFocusTest/ActivateFocusTest.html >> fails intermittently >> bug: https://bugs.openjdk.java.net/browse/JDK-8047961 >> Webrev: http://cr.openjdk.java.net/~mcherkas/8047961/webrev.00/ >> >> Thanks, >> Mikhail. >> >
