<AWT Dev> Please review changes in regression test /java/awt/TestArea/UsingWithMouse.java (second version of patch)

Pavel Tisnovsky ptisnovs at redhat.com
Tue Jan 26 03:57:05 PST 2010

Artem Ananiev wrote:
>> I've added synchronization of scrolling using method
>> SunToolkit.realSync(), it seems to works perfectly on RHEL x86_64 +
>> Gnome and Fedora 10 i386 + Gnome. Thanks Anthony Petrov for his tip!
> the problem looks really, really strange... Util.waitForIdle() is 
> implemented exactly the same way: as a call to realSync() - while you 
> wrote that calling waitForIdle() didn't help.
> Please, postpone your fix (don't integrate into AWT gate repository) and 
> check if Util.waitForIdle() does the job. realSync() is a low-level and 
> private Sun's API, so I'd prefer having it in a single place (in Util) 
> so we can change it easily, if necessary (e.g. when realSync() becomes 
> public).

Yes, you are right, Util.waitForIdle() simply calls realSync(). It is 
interesting - without this sync (waifForIdle/realSync, does not matter 
which I use) this test always fails on my machine (i.e. it does not 
selects all lines of TextArea), with sync included it fails "only" in 
circa 1/10 of its run. At this time, I don't plan to push this patch as 
it isn't 100% correct.

> Thanks,
> Artem
>> This patch can be applied to OpenJDK7 too.
>> Thanks in advance
>> Pavel Tisnovsky
>> Red Hat QA

More information about the awt-dev mailing list