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: [email protected]
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
>   
>

Reply via email to