[webkit-reviews] review canceled: [Bug 30442] Add helper methods to determine whether a transformation matrix is only scaled or translated : [Attachment 41383] Add a mapsRectToRect() helper method to TransformationMatrix (try 2)

[bugzilla-daemon at webkit.org]

Jakob Petsovits <jpetsovits at rim.com> has canceled Jakob Petsovits
<jpetsovits at rim.com>'s request for review:
Bug 30442: Add helper methods to determine whether a transformation matrix is
only scaled or translated
https://bugs.webkit.org/show_bug.cgi?id=30442

Attachment 41383: Add a mapsRectToRect() helper method to TransformationMatrix
(try 2)
https://bugs.webkit.org/attachment.cgi?id=41383&action=review

------- Additional Comments from Jakob Petsovits <jpetsovits at rim.com>
I had a talk with Simon and Chris on Skype, and it was determined that checking
the matrix would not be the best approach for the intended goal (which can be
paraphrased as "determine whether a transformation will return a rectilinear
quad in order to enable optimizations"). Instead of checking the transformation
itself, they suggested a new method FloatQuad::isRectilinear() to check the
output of that transformation, which is a more generic approach and can be used
in a couple more cases (e.g. also applies to rectangles turned by 90/270
degrees, and a few less common cases).

Here's the implementation; I tested with rectangles that are flipped and turned
by 90, 180 and 5 degrees, the latter will return false for isRectilinear()
while all of the others return true. Please review, hopefully this time it's
the way to go :)