Thanks for providing test coverage for 8152912. The changes look good!
Best,
Joe
On 5/5/2016 3:44 AM, Frank Yuan wrote:
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
