On 01.09.16 16:38, Semyon Sadetsky wrote:
If the test is correct why you tries to change it? The test found a
bug when we get more notifications than expected.
The issue was to count normal notifications from revalidation. Now the
counter starts to count after the layout is done.

But if layout was done, why we get any notifications? Should the check be (cnt != 0)?


I assume that each notification should be called when the size of
JViewport or View were changed?




On 23.08.16 11:59, Alexander Scherbatiy wrote:
The fix looks good to me.

Thanks,
Alexandr.

On 23/08/16 11:40, Semyon Sadetsky wrote:
Hello,

Please review fix for JDK9:

bug: https://bugs.openjdk.java.net/browse/JDK-8163167

webrev:
http://cr.openjdk.java.net/~ssadetsky/8163167/webrev.00/

The cause of this test bug is revalidating JScrollPane
layout may
take
various number of iterations on different L&Fs.

To fix the test the counter listener is added after the
layout
revalidation has been started.

--Semyon






















--
Best regards, Sergey.

Reply via email to