RFR: 8155039: Simplify code to setup SSLContextImpl and TrustManagerFactoryImpl

[Xuelei Fan]

Hi Claes,

What's the necessary to get all system properties for just one specific
one?  Can you explain more about the necessary to make the change?

Thanks,
Xuelei

On 6/8/2016 3:44 AM, Claes Redestad wrote:
> Hi,
> 
> there is some lingering concern that this and related changes make it
> that much easier to accidentally leak the system Properties object
> outside of core modules. By wrapping access to the system Properties
> object in a class residing in a non-exported package we disallow this at
> little to no cost:
> 
> http://cr.openjdk.java.net/~redestad/8155039/webrev.02/
> 
> /Claes
> 
> On 2016-05-12 15:37, Claes Redestad wrote:
>> Hi,
>>
>> the API this improvement depends on was updated to reflect more clearly
>> that this
>> is taking a privileged action:
>> https://bugs.openjdk.java.net/browse/JDK-8155775
>>
>> Here's the updated webrev:
>> http://cr.openjdk.java.net/~redestad/8155039/webrev.01/
>>
>> Thanks!
>>
>> /Claes
>>
>> On 2016-04-25 19:28, Claes Redestad wrote:
>>> Hi,
>>>
>>> SSLContextImpl and TrustManagerFactoryImpl has some setup code that
>>> could be
>>> simplified, getting rid of a couple of anonymous classes in the process.
>>>
>>> Webrev: http://cr.openjdk.java.net/~redestad/8155039/webrev.00
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8155039
>>>
>>> Alternatively we could remove OpenFileInputStreamAction instead since
>>> these two uses
>>> introduced here are actually the only active uses of this old
>>> convenience class.
>>>
>>> Thanks!
>>>
>>> /Claes
>>