Re: JEP 232 RFR: JDK-8065942 and JDK-8056179

2015-06-08 Thread Sean Mullan
On 06/05/2015 11:24 PM, Weijun Wang wrote: ServicePermission: We can also add a ServicePermission(String,int) like FilePermission has. Good point. Updated webrev: http://cr.openjdk.java.net/~mullan/webrevs/8065942-8056179/webrev.01/ Changes: - Fixed typo in SocketPermission: JDK-4301064

[9] Review request for 8085979: Make some DTLS feature functional tests work also for TLS protocol

2015-06-08 Thread Konstantin Shefov
Hello, Please review distribution of some DTLS feature tests to TLS protocol. Some DTLS tests may also be used to test the same functionality in TLS protocol and its versions. It is test only improvement. bug: https://bugs.openjdk.java.net/browse/JDK-8085979 webrev: http://cr.openjdk.java.net/

Re: JEP 232 RFR: JDK-8065942 and JDK-8056179

2015-06-08 Thread Weijun Wang
Great. Everything is fine. Thanks Max On 06/08/2015 10:24 PM, Sean Mullan wrote: On 06/05/2015 11:24 PM, Weijun Wang wrote: ServicePermission: We can also add a ServicePermission(String,int) like FilePermission has. Good point. Updated webrev: http://cr.openjdk.java.net/~mullan/webrevs/8065

Re: RFR 7191662: JCE providers should be located via ServiceLoader

2015-06-08 Thread Valerie Peng
What is the bug id for the image refresh change? Just so I can keep a watch and hold my changes for the time being. My webrev has a new regression test which compares the list of providers found by ServiceLoader and Security.getProviders() call. So, if the META-INF/services/java.security.Pro

Re: RFR 7191662: JCE providers should be located via ServiceLoader

2015-06-08 Thread Mandy Chung
JDK-8066492 Mandy On 06/08/2015 04:44 PM, Valerie Peng wrote: What is the bug id for the image refresh change? Just so I can keep a watch and hold my changes for the time being. My webrev has a new regression test which compares the list of providers found by ServiceLoader and Security.get