Mandy Chung wrote: > Alan, > > Here is the revised webrev: > http://cr.openjdk.java.net/~mchung/6959965/webrev.01/ Looks fine to me. Minor comment is that you might want to rename INTERNAL_OPTIONS_FILENAME given that you've renamed the file name. Thanks for expanding the test coverage for this. -Alan.