RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v2]

[Prasanta Sadhukhan]

On Tue, 1 Mar 2022 16:38:30 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix test comments
>
> test/jdk/javax/swing/JScrollPane/TestMouseWheelScroll.java line 48:
> 
>> 46:     static Point p;
>> 47:     static int width;
>> 48:     static int height;
> 
> These three should also be volatile, shouldn't they?

yes

> test/jdk/javax/swing/JScrollPane/TestMouseWheelScroll.java line 101:
> 
>> 99:                 robot.delay(1000);
>> 100:                 robot.mouseWheel(1);
>> 101:                 robot.delay(500);
> 
> You don't call `waitForIdle` or `delay` after the mouse click because it just selects an item but doesn't change the position, right?

added

-------------

PR: https://git.openjdk.java.net/jdk/pull/7585