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



Reply via email to