+1, but please rename the title of the bug to some better text, right now "what to fix?" looks cryptic.

On 14.09.16 20:07, Semyon Sadetsky wrote:
Looks good.

--Semyon


On 9/14/2016 7:56 PM, Ambarish Rapte wrote:

Hi,

                Please review this fix for JDK9,

                Webrev:
http://cr.openjdk.java.net/~arapte/8162102/webrev.00/
<http://cr.openjdk.java.net/%7Earapte/8162102/webrev.00/>

                Bug: https://bugs.openjdk.java.net/browse/JDK-8162102



Issue:


“/java/awt/AppContext/ApplicationThreadsStop/ApplicationThreadsStop.java/"
 Test fails with

Exception /java.security.AccessControlException: access denied
("java.util.PropertyPermission" "awt.robot.gtk" "read")/

All tests using security policy would fail with same exception.



Cause:

                Issue occurs when only a security policy is used.

                System property  /awt.robot.gtk/  accessing was added
as fix of https://bugs.openjdk.java.net/browse/JDK-8150954

Discussion mail:
http://mail.openjdk.java.net/pipermail/awt-dev/2016-April/010999.html



Fix:

                Added the getBoolean call inside a privileged block.





Regards,

Ambarish






--
Best regards, Sergey.

Reply via email to