Fix looks good. +1 on test renaming.
-Sundar On 5/12/2016 12:02 AM, Mandy Chung wrote: >> On May 11, 2016, at 11:00 AM, Jim Laskey (Oracle) <james.las...@oracle.com> >> wrote: >> >> http://cr.openjdk.java.net/~jlaskey/8156781/webrev/index.html >> https://bugs.openjdk.java.net/browse/JDK-8156781 > Looks okay. Since the plugin is renamed to OrderResourcesPlugin, maybe good > to rename it to OrderResourcesPluginTest. > > Mandy