<AWT Dev> [12] Review Request: 8210692 The "com.sun.awt.SecurityWarning" class can be dropped

[Philip Race]

Were people mis-reading this as java.desktop ?

Yes, its fine,  this was added under 
https://bugs.openjdk.java.net/browse/JDK-8055206
and we should have removed that when jdk.desktop was backed out
under https://bugs.openjdk.java.net/browse/JDK-8173409

-phil.

On 9/14/18, 11:41 AM, Sean Mullan wrote:
> On 9/13/18 7:52 PM, Stuart Marks wrote:
>> I'd guess that security-dev would have reviewers for the change to 
>> default.policy. Cc'd.
>
> The default.policy file change looks like this is not specifically 
> related to this change because the jdk.desktop module no longer 
> exists. I guess it is ok to include it with this change but I think 
> someone else (maybe Phil Race) should double-check that is ok.
>
> --Sean
>>
>> s'marks
>>
>> On 9/13/18 2:43 PM, Sergey Bylokhov wrote:
>>> Hello.
>>> Please review fix for jdk12.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8210692
>>> Webrev: http://cr.openjdk.java.net/~serb/8210692/webrev.00
>>> CSR: https://bugs.openjdk.java.net/browse/JDK-8210693
>>>
>>> The client code has a "com.sun.awt.SecurityWarning" class which at 
>>> some point in the past, JDK 6u10, was used as a kind of "public" API.
>>>
>>> During development of jdk9 it was considered that this API is not 
>>> supported and may be removed in the future: JDK-8051640.
>>> In jdk11 this class was marked as "forRemoval = true" (see 
>>> JDK-8205588) and can be dropped in 12.
>>>
>>> In the fix these things were removed:
>>>   - com/sun/awt/SecurityWarning.java
>>>   - The test for SecurityWarning class
>>>   - Test groups, because SecurityWarning is the latest class in 
>>> "com/sun/awt" package
>>>   - The reference to "com/sun/awt" in the "default.policy"
>>>
>>> (cc) core-libs-dev:
>>> I am not sure who is responsible to review the change in 
>>> "default.policy".
>>>
>>>