[8] Review request for 7172187 [macosx] JAWT native CALayer not positioned over Canvas (original) (raw)

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/viewbug.do?bugid=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