RFR: 6318027: BasicScrollBarUI does not disable timer when enclosing frame is disabled. [v5]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Mon Aug 19 02:25:29 UTC 2024
> 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..
Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
test typo
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/20346/files
- new: https://git.openjdk.org/jdk/pull/20346/files/526c6a17..de7ad6b7
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=20346&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=20346&range=03-04
Stats: 3 lines in 1 file changed: 0 ins; 1 del; 2 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
More information about the client-libs-dev
mailing list