Hi Frank, Looks good to me!
best regards, -- daniel On 07/12/16 03:15, Frank Yuan wrote:
Hi all Would you like to review http://cr.openjdk.java.net/~fyuan/8169948/webrev.00/? Bug: https://bugs.openjdk.java.net/browse/JDK-8169948 This test update is because of JDK-8169778 Add new public methods to get new instances of the JAXP factories builtin system-default implementations. DefaultFactoryWrapperTest.java tests a provider that return a custom factory instance wrapping the built-in system-default implementation. Thanks Frank