Looks good to me. Ripping out code for a fix is a good thing. -- Scott
On May 24, 2012, at 5:23 PM, Leonid Romanov wrote: > Hi, > Please review a fix for 7159381: [macosx] Dock Icon defaults to Generic Java > Application Category. The problem here is that we ignore the fact that for > the bundled app its icon might be specified via Info.plist file. In this case > OS X sets the icon for us, so we don't have to do anything. > > Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7159381 > Webrev: http://cr.openjdk.java.net/~leonidr/7159381/webrev.00/ > > Thanks, > Leonid. >