[7u40] Request for phase 2 review and approval for 8019979

[Dalibor Topic]

On 7/11/13 3:28 PM, Seán Coffey wrote:
> 8019979  : Replace CheckPackageAccess test with better one from closed repo
> 
> I'd like to push this testcase enhancement to 7u40. It was pushed to jdk8 TL repo earlier in the week. I've run tests via JPRT and saw no issues. The package list in testcase is slightly different to that in jdk8 given the package differences that exist between 7u & 8.
> 
> Sean - can I ask you to review the change again ? A second review would be welcome also.
> 
> bug report : http://bugs.sun.com/view_bug.do?bug_id=8019979
> jdk8 review : http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-July/018811.html
> 7u40 webrev : http://cr.openjdk.java.net/~coffeys/webrev.8019979.7u40/webrev/

Looks good, approved!

cheers,
dalibor topic



-- 
Oracle <http://www.oracle.com>
Dalibor Topic | Principal Product Manager
Phone: +494089091214 <tel:+494089091214> | Mobile: +491737185961 <tel:+491737185961>
Oracle Java Platform Group

ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg

ORACLE Deutschland B.V. & Co. KG
Hauptverwaltung: Riesstr. 25, D-80992 München
Registergericht: Amtsgericht München, HRA 95603
Geschäftsführer: Jürgen Kunz

Komplementärin: ORACLE Deutschland Verwaltung B.V.
Hertogswetering 163/167, 3543 AS Utrecht, Niederlande
Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697
Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher

Green Oracle <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment