<AWT Dev> Please review changes in regression test /java/awt/TestArea/UsingWithMouse.java (second version of patch)
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
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.
>> This patch can be applied to OpenJDK7 too.
>> Thanks in advance
>> Pavel Tisnovsky
>> Red Hat QA
More information about the awt-dev