<AWT Dev> [awt-dev] Request for Approval and Review: JDK-8132376: [TEST_BUG] Add @requires os.family to the client tests with access to internal OS-specific API

[Renjith Alexander]

Hi All,

Please review the updated fix for Bug JDK-8132376:

Webrev: http://cr.openjdk.java.net/~sgupta/8132376/webrev.03/

The Webrev for diff between files in closed(which are moved to open in the above webrev):

http://cr.openjdk.java.net/~sgupta/8132376/webrev.02/

Regards,
Renjith.



-----Original Message-----
From: Alexander Scherbatiy 
Sent: Monday, August 10, 2015 7:24 PM
To: Renjith Alexander
Cc: awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [awt-dev] Request for Approval and Review: JDK-8132376: [TEST_BUG] Add @requires os.family to the client tests with access to internal OS-specific API

On 8/7/2015 1:56 PM, Renjith Alexander wrote:
>
> Hi all,
>
> Please review the fix for the bug:  JDK-8132376: Add @requires 
> os.family to the client tests with access to internal OS-specific API
>
> BUG ID -*https://bugs.openjdk.java.net/browse/JDK-8132376*
>   
> The webrev is:http://cr.openjdk.java.net/~sgupta/8132376/webrev.01/  
> <http://cr.openjdk.java.net/%7Esgupta/8132376/webrev.01/>

     - the copyright needs to be added to the new files
     - the unnecessary staff for manual execution can be removed from tests which are made automated

    Thanks,
    Alexandr.

>   
>   
> A couple of files(which are shown as new in the previous webrev) have been moved from ‘closed’ to ‘open’. The difference for those files in the closed repo can be found in the webrev:
>   
> http://cr.openjdk.java.net/~sgupta/8132376/webrev.00/  
> <http://cr.openjdk.java.net/%7Esgupta/8132376/webrev.00/>
>
> --
> Regards,
> Renjith Alexander
>   
>