Issue is
BasicScrollBarUI.ArrowButtonListener starts a timer in mousePressed(), and 
stops it in mouseReleased(). If the frame containing the scrollbar is disabled 
between the MOUSE_PRESSED and the MOUSE_RELEASED events, the mouseReleased() 
method is never called. If the frame is then re-enabled, the still-running 
timer causes it to scroll all the way to the end.
Fix is to check if 
[ArrowButtonListener.handledEvent](https://github.com/openjdk/jdk/blame/ee839b7f0ebe471d3877cddd2c87019ccb8ee5ae/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicScrollBarUI.java#L1567)
 is still set when ActionEvent is processed then stop the timer and reset this 
variable.

CI testing is green and also SwingSet2 JScrollPane scrolling with this 
modification..

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

Commit messages:
 - Test format
 - Test format
 - Test format
 - 6318027: BasicScrollBarUI does not disable timer when enclosing frame is 
disabled.

Changes: https://git.openjdk.org/jdk/pull/20346/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=20346&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-6318027
  Stats: 192 lines in 2 files changed: 192 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/20346.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20346/head:pull/20346

PR: https://git.openjdk.org/jdk/pull/20346

Reply via email to