--- old/src/jdk.accessibility/windows/classes/com/sun/java/accessibility/internal/AccessBridge.java 2020-02-29 15:33:03.766456752 +0530 +++ new/src/jdk.accessibility/windows/classes/com/sun/java/accessibility/internal/AccessBridge.java 2020-02-29 15:33:03.454456733 +0530 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2005, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2005, 2020, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -1457,6 +1457,38 @@ return null; } + private int getNonVisibleChildrenCountTillIndex(AccessibleContext parentAC, int index) { + if (parentAC != null && index >= 0 && index < parentAC.getAccessibleChildrenCount()) { + int nonVisibleChildrenCount = 0; + for (int i = 0; i <= index; i++) { + if (!parentAC.getAccessibleChild(i).getAccessibleContext().getAccessibleStateSet().contains(AccessibleState.VISIBLE)) { + nonVisibleChildrenCount++; + } + } + return nonVisibleChildrenCount; + } + return 0; + } + + private Accessible getVisibleChildAtIndex(AccessibleContext parentAC, int index) { + if (parentAC != null && index >= 0 && index < parentAC.getAccessibleChildrenCount()) { + int visibleIndex = -1; + int childrenCount = parentAC.getAccessibleChildrenCount(); + for (int i = 0; i <= childrenCount; i++) { + Accessible child = parentAC.getAccessibleChild(i); + if (child != null) { + AccessibleContext ac = child.getAccessibleContext(); + if (ac != null && ac.getAccessibleStateSet().contains(AccessibleState.VISIBLE)) { + visibleIndex++; + } + if (visibleIndex == index) { + return child; + } + } + } + } + return null; + } /** * returns the AccessibleParent from an AccessibleContext */ @@ -1487,7 +1519,12 @@ return InvocationUtils.invokeAndWait(new Callable() { @Override public Integer call() throws Exception { - return ac.getAccessibleIndexInParent(); + int indexInParent = ac.getAccessibleIndexInParent(); + Accessible parent = ac.getAccessibleParent(); + if (parent != null) { + indexInParent -= getNonVisibleChildrenCountTillIndex(parent.getAccessibleContext(), indexInParent); + } + return indexInParent; } }, ac); } @@ -1501,7 +1538,8 @@ return InvocationUtils.invokeAndWait(new Callable() { @Override public Integer call() throws Exception { - return ac.getAccessibleChildrenCount(); + int childrenCount = ac.getAccessibleChildrenCount(); + return childrenCount - getNonVisibleChildrenCountTillIndex(ac, childrenCount - 1); } }, ac); } @@ -1537,7 +1575,7 @@ return InvocationUtils.invokeAndWait(new Callable() { @Override public AccessibleContext call() throws Exception { - Accessible a = ac.getAccessibleChild(index); + Accessible a = getVisibleChildAtIndex(ac, index); if (a != null) { return a.getAccessibleContext(); } @@ -3517,7 +3555,11 @@ AccessibleRelation[] relations = ars.toArray(); if (relations != null && i >= 0 && i < relations.length) { Object[] targets = relations[i].getTarget(); - return targets.length; + if (targets != null) { + int targetCount = targets.length - + getNonVisibleTargetCountTillIndex(targets, targets.length - 1); + return targetCount; + } } } } @@ -3543,7 +3585,7 @@ if (relations != null && i >= 0 && i < relations.length) { Object[] targets = relations[i].getTarget(); if (targets != null && j >= 0 & j < targets.length) { - Object o = targets[j]; + Object o = getVisibleTargetAtIndex(targets, j); if (o instanceof Accessible) { return ((Accessible) o).getAccessibleContext(); } @@ -3556,6 +3598,40 @@ }, ac); } + private Object getVisibleTargetAtIndex(Object[] targets, int index) { + if (index >= 0 && index < targets.length) { + int visibleTargetIndex = -1; + for (int i = 0; i < targets.length; i++) { + if (targets[i] instanceof Accessible) { + AccessibleContext ac = ((Accessible) targets[i]).getAccessibleContext(); + if (ac != null && ac.getAccessibleStateSet().contains(AccessibleState.VISIBLE)) { + visibleTargetIndex++; + } + if (visibleTargetIndex == index) { + return targets[i]; + } + } + } + } + return null; + } + + private int getNonVisibleTargetCountTillIndex(Object[] targets, int index) { + if (index >= 0 && index < targets.length) { + int nonVisibleTargetsCount = 0; + for (int i = 0; i <= index; i++) { + if (targets[i] instanceof Accessible) { + AccessibleContext ac = ((Accessible) targets[i]).getAccessibleContext(); + if (ac != null && !ac.getAccessibleStateSet().contains(AccessibleState.VISIBLE)) { + nonVisibleTargetsCount++; + } + } + } + return nonVisibleTargetsCount; + } + return 0; + } + // ========= AccessibleHypertext ========= private Map hyperTextContextMap = new WeakHashMap<>();