Hello, Alexander. The fix looks fine, just the frame and dialogs fields should be volatile. The new webrev is not needed for this little change.
With best regards. Petr. On 27 окт. 2014 г., at 14:26, alexander stepanov <[email protected]> wrote: > Hello, > > Could please anyone review this old request? > > Thanks, > Alexander > > On 25.09.2014 17:10, alexander stepanov wrote: >> Sorry, just a reminder. >> >> Thanks, >> Alexander >> >> On 11.09.2014 18:27, alexander stepanov wrote: >>> Could anyone review this old request? >>> >>> Thanks, >>> Alexander >>> >>> On 15.08.2014 12:19, alexander stepanov wrote: >>>> Hello, >>>> >>>> Could you please review the fix for >>>> https://bugs.openjdk.java.net/browse/JDK-8054358 >>>> >>>> webrev: >>>> http://cr.openjdk.java.net/~avstepan/8054358/ >>>> >>>> This is the next portion of functional AWT tests prepared for migration to >>>> OpenJDK repository. >>>> >>>> The tests were checked on Ubuntu 14.04 Linux, Solaris 11, Mac OS X 10.8.5 >>>> and Windows 7. >>>> >>>> Thanks, >>>> Alexander >>> >> >
