Hi
Would you like to review http://cr.openjdk.java.net/~fyuan/8156119/webrev.00/? Bug: https://bugs.openjdk.java.net/browse/JDK-8156119 This change is to add/update some tests for verifying JDK-8152912: SAX XMLReaderFactory needs to be ServiceLoader compliant, see Joe's RFR mail thread http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-May/040731.html XMLReaderFactoryTest.java tests 1. XMLReaderFactory can create XMLReader as legacy way, and fix JDK-8015099 Classloading boundary crossing in Java 7 2. XMLReaderFactory can load XMLReader impl by service resource I also updated existing tests to verify XMLReaderFactory can load XMLReader impl from provider module. Thanks, Frank