On Aug 23, 2010, at 4:33 AM, Maciej Stachowiak wrote:

> 
> 
> - tx, ty are the origin of the parent's coordinate system relative to the 
> origin for its layer. When a layer paints, it establishes a CTM such that its 
> own origin is 0, 0 (I think).
> 

They are relative to a painting root, which will basically either be the 
document or a compositing layer.

> The more obvious way, given the variable names, would be to make x(), y() an 
> IntPoint, with a name like originInParentCoordinates()

It already is an IntPoint internally.  There's already a method to access it as 
a point.

IntPoint location() const

> (hopefully less verbose, but you get the idea). tx, ty could be named 
> parentOffsetFromLayerCoordinates or something. This seems to be the intent of 
> the names - that x,y is a point and tx, ty is a translation. But this doesn't 
> work in point/size logic. You repeatedly add x(), y() to tx, ty to get a new 
> tx, ty. But that means you're adding a point to a size and expecting to get a 
> new size - but that's not how it works.
> 

I think a helper method that does the right thing solves this problem (rather 
than having to flip what x/y and tx/ty mean just to do some math operation).

dave

_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to