Those changes look fine to me.
On 27 Aug 2013, at 14:34, Sean Mullan wrote: > Hi, > > Could you please review my fix for 8019830: > > webrev: http://cr.openjdk.java.net/~mullan/webrevs/8019830/webrev.00/ > > The bug is not yet available on bugs.sun.com, so here is the description of > the bug: > > com.sun.media.sound should be added to the list of restricted packages in JDK > 8. This means applications running under a SecurityManager will not be able > to access classes in this package hierarchy unless granted explicit > permission. com.sun.media.sound is an internal, unsupported package and is > not meant to be used by external applications. > > Thanks, > Sean
