<AWT Dev> [7u10] Review request for 7172187 [macosx] JAWT native CALayer not positioned over Canvas

Anthony Petrov anthony.petrov at oracle.com
Tue Sep 11 05:57:16 PDT 2012


Looks good.

--
best regards,
Anthony

On 9/6/2012 3:29 PM, Sergey Bylokhov wrote:
> Hi Everyone,
> Please review the fix. This is direct backport from jdk 8.
> 
> Description of main changes:
> 
> CPlatformComponent.java:
>     -Unused peer and target references were removed.
>     -Wrong return value for nativeSetBounds() was changed from long to 
> void.
> AWTSurfaceLayers.m:
>     -layer.bounds was replaced by layer.frame, because layer.bounds 
> doesn't honour x and y position of the layer.
>     -Wrong position inversion was removed.
> Also related fields were marked as volatile, if they were used in the 
> different threads + javadoc cleanup.
> 
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7172187
> Webrev can be found at: http://cr.openjdk.java.net/~serb/7172187/webrev.00
> jdk8 changeset: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/65d874d16d59
> 


More information about the macosx-port-dev mailing list