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

Artem Ananiev artem.ananiev at oracle.com
Tue Aug 14 07:39:11 PDT 2012


Looks fine.

Thanks,

Artem

On 8/14/2012 6:30 PM, Sergey Bylokhov wrote:
> Hi Everyone,
> Please review the fix.
> 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
>
> --
> Best regards, Sergey.
>



More information about the awt-dev mailing list