[8] Review request for 7172187 [macosx] JAWT native CALayer not positioned over Canvas (original) (raw)
Sergey Bylokhov Sergey.Bylokhov at oracle.com
Tue Aug 14 07:30:07 PDT 2012
- Previous message: [8] Review request for 2226249: [macosx] Java processes on Mac should not use default Apple icon
- Next message: [8] Review request for 7172187 [macosx] JAWT native CALayer not positioned over Canvas
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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.
-------------- next part -------------- An HTML attachment was scrubbed... URL: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20120814/49fa10f3/attachment.html
- Previous message: [8] Review request for 2226249: [macosx] Java processes on Mac should not use default Apple icon
- Next message: [8] Review request for 7172187 [macosx] JAWT native CALayer not positioned over Canvas
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]