365886 - Implement canvas transform() and setTransform()

Status: RESOLVED FIXED

(Core :: Graphics: Canvas2D, enhancement)

Description

[Takeshi Kurosawa :takenspc]

Current Web Applications 1.0 introduces transform(m11, m12, m21, m22, dx, dy) and setTransform(m11, m12, m21, m22, dx, dy) into the canvas 2d context.

The rhino canvas implemented them already.
http://rhino-canvas.sourceforge.net/

We should implement them.

Comment 1

[Takeshi Kurosawa :takenspc]

Attachment: Patch rv.1.0

A patch as per the spec.

Both methods uses the follow matrix:

transform(m11, m12, m21, m22, dx, dy) <->

m11 	m12 	dx
m21 	m22 	dy
0 	0 	1

Comment 2

[Takeshi Kurosawa :takenspc]

Attachment: Test case

A test case as per the updated spec.

Now, the matrix is below.
transform(m11, m12, m21, m22, dx, dy) <->

m11     m21     dx
m12     m22     dy
0       0       1

Comment 3

[Takeshi Kurosawa :takenspc]

Attachment: Patch rv.1.1

A new patch followed the updated spec

Comment 4

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Comment on attachment 251788 [details] [diff] [review]
Patch rv.1.1

Looks good; do you have cvs commit access?  If not, I'll get this checked in.

Comment 5

[Takeshi Kurosawa :takenspc]

(In reply to comment #4)
> (From update of attachment 251788 [details] [diff] [review])
> Looks good; do you have cvs commit access?  If not, I'll get this checked in.
> 

No, I don't. I beg of you to do so.

Comment 6

[Takeshi Kurosawa :takenspc]

Attachment: Patch rv.1.1.1

change the uuid of |nsIDOMCanvasRenderingContext2D|.

Comment 7

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

checked-in, thank you Taken-san.