Reviewer needed - patch which fixes class SunToolkit for IcedTea6-1.10

[Dr Andrew John Hughes]

On 10:34 Mon 09 May     , Omair Majid wrote:
> On 03/31/2011 10:43 AM, Dr Andrew John Hughes wrote:
> > On 12:42 Thu 31 Mar     , Pavel Tisnovsky wrote:
> >> Hi all,
> >>
> >> I'd like to backport SunToolkit patch to IcedTea6-1.10 (it's already
> >> pushed to IcedTea6 HEAD). Can anybody please review it?
> >>
> >> 2011-03-31  Pavel Tisnovsky<ptisnovs at redhat.com>
> >>
> >>      * Makefile.am: Add new patch.
> >>      * NEWS: Updated (added info about new fix).
> >>      * patches/mark_sun_toolkit_privileged_code.patch:
> >>      Patch which fixes the AccessControlException while
> >>      working with Color class in a PropertyEditor.
> >>
> >> Pavel T.
> >
> > Backport looks fine, assuming it builds.  Please commit.
> >
> > Was there any response from Oracle as to the bug ID?  I didn't see anything.
> >
> 
> I am going to guess it's 6599601 [1]. The bug lists that it was fixed in 
> b21 (another source [2]), and the diff between openjdk7 b20 and b21 
> seems to agree [3]. May I move this patch to 
> patches/openjdk/6599601-mark_sun_toolkit_privileged_code.patch?
> 

Yes, and you now have permission from Kelly to push it to OpenJDK6.

> Cheers,
> Omair
> 
> [1] http://bugs.sun.com/view_bug.do?bug_id=6599601
> [2] http://download.java.net/jdk7/changes/jdk7-b21.html
> [3] 
> http://icedtea.classpath.org/hg/openjdk/diff/e2018f79de77/j2se/src/share/classes/sun/awt/SunToolkit.java

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37