Re: RFR (JAXP) JDK-8156119: Update ServiceProviderTest for XMLReaderFactory (tests for 8152912)

2016-05-05 Thread huizhe wang
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/upd

RFR (JAXP) JDK-8156119: Update ServiceProviderTest for XMLReaderFactory

2016-05-05 Thread Frank Yuan
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 h