--- old/src/java.desktop/share/classes/javax/swing/JList.java 2017-04-13 16:07:01.184050100 +0300 +++ new/src/java.desktop/share/classes/javax/swing/JList.java 2017-04-13 16:07:00.457008500 +0300 @@ -3395,15 +3395,18 @@ } public AccessibleSelection getAccessibleSelection() { - return getCurrentAccessibleContext().getAccessibleSelection(); + AccessibleContext ac = getCurrentAccessibleContext(); + return ac != null ? ac.getAccessibleSelection() : null; } public AccessibleText getAccessibleText() { - return getCurrentAccessibleContext().getAccessibleText(); + AccessibleContext ac = getCurrentAccessibleContext(); + return ac != null ? ac.getAccessibleText() : null; } public AccessibleValue getAccessibleValue() { - return getCurrentAccessibleContext().getAccessibleValue(); + AccessibleContext ac = getCurrentAccessibleContext(); + return ac != null ? ac.getAccessibleValue() : null; } --- /dev/null 2017-04-13 16:07:06.000000000 +0300 +++ new/test/javax/accessibility/JList/AccessibleJListChildNPETest.java 2017-04-13 16:07:05.245282400 +0300 @@ -0,0 +1,99 @@ +/* +* Copyright (c) 2017, 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 +* under the terms of the GNU General Public License version 2 only, as +* published by the Free Software Foundation. +* +* This code is distributed in the hope that it will be useful, but WITHOUT +* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or +* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +* version 2 for more details (a copy is included in the LICENSE file that +* accompanied this code). +* +* You should have received a copy of the GNU General Public License version +* 2 along with this work; if not, write to the Free Software Foundation, +* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. +* +* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA +* or visit www.oracle.com if you need additional information or have any +* questions. +*/ + +import java.lang.reflect.InvocationTargetException; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.List; + +import javax.accessibility.Accessible; +import javax.accessibility.AccessibleContext; +import javax.swing.*; + +/* @test + @bug 8076249 + @summary NPE in AccessBridge while editing JList model + @author Mikhail Cherkasov + @run main AccessibleJListChildNPETest +*/ +public class AccessibleJListChildNPETest { + + private static String[] model = { "1", "2", "3", "4", "5", "6" }; + private static JList list; + + public static void main(String[] args) throws InvocationTargetException, InterruptedException { + SwingUtilities.invokeAndWait(new Runnable() { + @Override + public void run() { + JFrame frame = new JFrame(); + frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE); + final MyModel dataModel = new MyModel(Arrays.asList(model)); + list = new JList<>(dataModel); + frame.getContentPane().add(list); + frame.pack(); + frame.setVisible(true); + + } + }); + + SwingUtilities.invokeAndWait(new Runnable() { + @Override + public void run() { + AccessibleContext ac = list.getAccessibleContext(); + MyModel model = (MyModel)list.getModel(); + Accessible accessibleChild = ac.getAccessibleChild(model.getSize()-1); + model.removeFirst(); + accessibleChild.getAccessibleContext().getAccessibleSelection(); + accessibleChild.getAccessibleContext().getAccessibleText(); + accessibleChild.getAccessibleContext().getAccessibleValue(); + } + }); + } + + protected static class MyModel extends AbstractListModel { + private List items = new ArrayList<>(); + + MyModel(final List newItems) { + super(); + items.addAll(newItems); + fireIntervalAdded(this, 0, getSize() - 1); + } + + void removeFirst() { + if(getSize() > 0) { + items.remove(0); + fireIntervalRemoved(this, 0, 0); + } + } + + @Override + public int getSize() { + return items.size(); + } + + @Override + public String getElementAt(int index) { + return items.get(index); + } + } +}