<Swing Dev> Review request for 8025430: [TEST_BUG] javax/swing/JEditorPane/5076514/bug5076514.java failed since jdk8b108

[Sergey Bylokhov]

looks fine.

On 14.04.16 17:02, prasanta sadhukhan wrote:
> ok. looks good to me.
>
> Regards
> Prasanta
> On 4/14/2016 1:59 PM, Muneer Kolarkunnu wrote:
>>
>> I added the @run tag and updated the review:
>> http://cr.openjdk.java.net/~ntv/muneer/8025430/webrev.01/
>> <http://cr.openjdk.java.net/%7Entv/muneer/8025430/webrev.01/>
>>
>> I tested with latest jtreg with and without @run tag, both are working.
>>
>> But there is a difference in the logs in the following line:
>>
>> *Without @run flag:*
>>
>> command: main bug5076514
>>
>> reason: Assumed action based on file name: run main bug5076514
>>
>> *With @run flag:*
>>
>> command: main bug5076514
>> reason: User specified action: run main bug5076514
>>
>> Regards,
>>
>> Muneer
>>
>> *From:*prasanta sadhukhan
>> *Sent:* Wednesday, April 13, 2016 7:49 PM
>> *To:* Muneer Kolarkunnu; swing-dev at openjdk.java.net
>> *Subject:* Re: Review request for 8025430: [TEST_BUG]
>> javax/swing/JEditorPane/5076514/bug5076514.java failed since jdk8b108
>>
>> You need to add
>> @run main <test> for latest jtreg to run. Have you run against latest
>> jtreg?
>>
>> Regards
>> Prasanta
>>
>> On 4/13/2016 6:10 PM, Muneer Kolarkunnu wrote:
>>
>>     Hi All,
>>
>>                     Please review the fix for test bug 8025430,
>>
>>                     Webrev:
>>     <http://cr.openjdk.java.net/%7Entv/muneer/8025430/webrev.00/>http://cr.openjdk.java.net/~ntv/muneer/8025430/webrev.00/
>>
>>                     Bug: https://bugs.openjdk.java.net/browse/JDK-8025430
>>
>>     Issue:
>>
>>     It is a test bug, API checkSystemClipboardAccess() is deprecated.
>>     Deprecated message from java doc for your reference : "The
>>     dependency on AWTPermission creates an impediment to future
>>     modularization of the Java platform. Users of this method should
>>     instead invoke checkPermission(java.security.Permission) directly.
>>     This method will be changed in a future release to check the
>>     permission java.security.AllPermission."
>>
>>     Fix:
>>
>>                   API  checkPermission(java.security.Permission) is
>>     used instead of checkSystemClipboardAccess()
>>
>>     Regards,
>>
>>     Muneer
>>
>


-- 
Best regards, Sergey.