[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