Re: Code review: 8010464: Evolve java networking same origin policy

2013-05-12 Thread Alan Bateman
On 10/05/2013 12:34, Michael McMahon wrote: Hi, This is the webrev for the HttpURLPermission addition. As well as the new permission class, the change includes the use of the permission in java.net.HttpURLConnection. The code basically checks for a HttpURLPermission in plainConnect(), getInputS

hg: jdk8/tl/jdk: 7021870: GzipInputStream closes underlying stream during reading

2013-05-12 Thread dmitry . degrave
Changeset: 90f715cceaae Author:dmeetry Date: 2013-05-10 23:56 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/90f715cceaae 7021870: GzipInputStream closes underlying stream during reading Reviewed-by: mduigou Contributed-by: ivan.gerasi...@oracle.com ! src/share/classes/java/

Re: Code review: 8010464: Evolve java networking same origin policy

2013-05-12 Thread Dmitry Samersoff
Michael, It might be better to narrow permissions right now with code like below: private static AccessControlContext withPermissions(Permission ... perms){ Permissions col = new Permissions(); for (Permission thePerm : perms ) { col.add(thePerm); } final ProtectionDomain pd = new

hg: jdk8/tl/langtools: 8014363: javac test class ToolTester handles classpath incorrectly

2013-05-12 Thread jonathan . gibbons
Changeset: e39669aea0bd Author:jjg Date: 2013-05-12 18:18 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/e39669aea0bd 8014363: javac test class ToolTester handles classpath incorrectly Reviewed-by: ksrini ! test/tools/javac/api/6406133/T6406133.java ! test/tools/javac/