The fix looks fine to me. One thing. The test is
in closed just because no one got round to moving it to open.
At least I don't see any reason it can't be opened up.
Now would be a good time and it can be included with this
changeset.
Don't forget to tag it with this additional bug ID.
-phil.
Roman Kennke wrote:
Hi again,
is this ok for committing?
Thanks, Roman
Am Donnerstag, den 15.10.2009, 15:22 +0200 schrieb Roman Kennke:
Hi there,
Here's a fix for Bug#6888734. One regression test was failing with an
AccessControlException in new FontManager code. We need to move
Class.forName() and newInstance() call into the privileged block. Webrev
here:
http://cr.openjdk.java.net/~rkennke/6888734/webrev.00/
Good? Comments?
/Roman
