<Swing Dev> [9] Review request for 8163167: [PIT] javax/swing/JTextArea/ScrollbarFlicker/ScrollFlickerTest.java always fail
Semyon Sadetsky
semyon.sadetsky at oracle.com
Mon Oct 3 09:35:37 UTC 2016
On 9/30/2016 5:53 PM, Sergey Bylokhov wrote:
> 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)?
Okay. I have changed it to zero:
http://cr.openjdk.java.net/~ssadetsky/8163167/webrev.01/
>
>>>
>>>>> 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
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>>
>>
>
>
More information about the swing-dev
mailing list