[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: RFR: 8282857: Create a regression test for JDK-4702690 [v2]
From: Sergey Bylokhov <serb () openjdk ! java ! net>
Date: 2022-03-29 22:46:39
Message-ID: Daj1h1XYMiZtVg6Xds2qF5j8zNcmkPFK3-SRDOO60lY=.8db40935-697a-4209-9c58-2320bd6feb88 () github ! com
[Download RAW message or body]
On Fri, 11 Mar 2022 15:46:32 GMT, Srinivas Mandalika <smandalika@openjdk.org> wrote:
> > Create a regression test for \
> > [JDK-4702690](https://bugs.openjdk.java.net/browse/JDK-4702690)
> > In many cases in Swing it is possible to easily programatically determine that a \
> > JScrollBar (or two) is scrolling some JPanel (the cannonical case is a \
> > JScrollPane). In these cases, when accessibility support is instantiated (e.g. \
> > the AccessibleJScrollBar is created), a Controller_For and Controled_By relation \
> > should be instantiated between the AccessibleJScrollBar and the AccessibleJPanel \
> > that the JScrollBar and JPanel are associated with.
> > This allows various assistive technologies, especially voice-recognition \
> > technologies, to better interact with scrolling items.
> > The test put up validates that the target object for these \
> > properties(CONTROLLED_BY, CONTROLLED_FOR) are set appropriately for JScrollPane \
> > and JScrollBar. This review is for migrating tests from a closed test suite to \
> > open.
>
> Srinivas Mandalika has updated the pull request incrementally with one additional \
> commit since the last revision:
> Added @bug keyword
test/jdk/javax/accessibility/4702690/JScrollPaneAccessibleRelationsTest.java line 63:
> 61: SwingUtilities.invokeAndWait(() -> createGUI());
> 62:
> 63: Object[] jScrollPaneTarget = jScrollPane.getAccessibleContext()
The jScrollPane, horizontalScrollBar, verticalScrollBar are all Swing components so \
should be accessed on the EDT.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7753
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic