Summary: | [OpenVG] Convert from TransformationMatrix to AffineTransform | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jakob Petsovits <jpetsovits> | ||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | ||||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Other | ||||||||
OS: | OS X 10.5 | ||||||||
Bug Depends on: | 35444 | ||||||||
Bug Blocks: | 33987, 34366 | ||||||||
Attachments: |
|
Description
Jakob Petsovits
2010-02-26 12:34:56 PST
Created attachment 49615 [details]
Patch
krit's commit changed most of the graphics platform layer to operate with the new AffineTransform instead of TransformationMatrix. That patch did not, however, update the OpenVG backend - which is fair enough, as there's currently no publicly available way to build it. This patch converts the already upstreamed parts of the OpenVG backend to use AffineTransform. krit said he'd get mapQuad() into AffineTransform too (that was originally left out), until that happens, I've got a private static function doing the same thing. We can clean it out afterwards. Please review! Comment on attachment 49615 [details] Patch Patch looks fine, r=me. > diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog > index cd85ac2..d539832 100644 > --- a/WebCore/ChangeLog > +++ b/WebCore/ChangeLog > @@ -1,3 +1,36 @@ > +2010-02-26 Jakob Petsovits <jpetsovits@rim.com> > + > + Reviewed by NOBODY (OOPS!). > + > + [OpenVG] Convert from TransformationMatrix to AffineTransform. > + the ChangeLog is missing a bug number and a good explainatain to why this is done. It shouldn't be necessary to look in the bug to get that information :-) Please fix before landing. Created attachment 49627 [details]
Patch
New patch (above), with bug URL in the commit message and private mapQuad() copy removed, instead we depend on bug 35444 to add mapQuad() directly to AffineTransform. Yay. Comment on attachment 49627 [details]
Patch
Great, r=me.
Committed r55369: <http://trac.webkit.org/changeset/55369> |