Title: [108447] trunk/Source/WebCore
- Revision
- 108447
- Author
- [email protected]
- Date
- 2012-02-21 22:29:33 -0800 (Tue, 21 Feb 2012)
Log Message
2D Canvas setTransform makes unnecessary matrix mults
https://bugs.webkit.org/show_bug.cgi?id=79190
Reviewed by Dan Bernstein.
No new tests - plenty of current tests using setTransform.
* html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::setTransform):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (108446 => 108447)
--- trunk/Source/WebCore/ChangeLog 2012-02-22 05:46:08 UTC (rev 108446)
+++ trunk/Source/WebCore/ChangeLog 2012-02-22 06:29:33 UTC (rev 108447)
@@ -1,3 +1,15 @@
+2012-02-21 Matthew Delaney <[email protected]>
+
+ 2D Canvas setTransform makes unnecessary matrix mults
+ https://bugs.webkit.org/show_bug.cgi?id=79190
+
+ Reviewed by Dan Bernstein.
+
+ No new tests - plenty of current tests using setTransform.
+
+ * html/canvas/CanvasRenderingContext2D.cpp:
+ (WebCore::CanvasRenderingContext2D::setTransform):
+
2012-02-21 MORITA Hajime <[email protected]>
INPUT_SPEECH should be implemented as a PageSupplement.
Modified: trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp (108446 => 108447)
--- trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp 2012-02-22 05:46:08 UTC (rev 108446)
+++ trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp 2012-02-22 06:29:33 UTC (rev 108447)
@@ -669,9 +669,8 @@
AffineTransform ctm = state().m_transform;
if (!ctm.isInvertible())
return;
- c->concatCTM(c->getCTM().inverse());
- c->concatCTM(canvas()->baseTransform());
- state().m_transform = ctm.inverse() * state().m_transform;
+ c->setCTM(canvas()->baseTransform());
+ state().m_transform = AffineTransform();
m_path.transform(ctm);
state().m_invertibleCTM = true;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
