1294360 - Remove CanvasRenderingContext2D::mozCurrentTransform/mozCurrentTransformInverse

Status: RESOLVED FIXED

(Core :: Graphics: Canvas2D, task, P3)

Comment 1

[Mathew Hodson]

Do you think we could disable these by default now? We added telemetry in bug 1765063.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

The use counter is at about 0.05, which is not too low. But doing a quick GitHub search I haven't found anything that would break... It seems mostly polyfills and so on.

So yeah, we can give this a shot. Thanks!

Comment 3

[Mathew Hodson]

Do you think it's safe to remove this code now? It was disabled by default in Firefox 106.

Comment 4

[Emilio Cobos Álvarez (:emilio)]

Probably worth waiting to the next nighly cycle but yeah, should be fine to remove :)

Comment 5

[Mathew Hodson]

Attachment: Bug 1294360 - Remove mozCurrentTransform. r?emilio

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/183400ded7d4
Remove mozCurrentTransform. r=emilio

Comment 7

[Narcis Beleuzu [:NarcisB]]

Backed out for bustages on CanvasRenderingContext2D.cpp

Backout link: https://hg.mozilla.org/integration/autoland/rev/3b21fbb7ca8df32cf62c876be21072f4156ec7ab
Log link: https://treeherder.mozilla.org/logviewer?job_id=395361569&repo=autoland&lineNumber=9994

Comment 8

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

When relanding, consider keeping the // Mozilla-specific stuff line in the IDL file for the remaining Mozilla-specific stuff

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dab1bbedb7c2
Remove mozCurrentTransform. r=emilio

Comment 10

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/dab1bbedb7c2

Comment 11

[Mathew Hodson]

I created bug 1799319, which adds a comment in the IDL file for the remaining mozTextStyle.

Comment 12

[Ruth John]

As docs have already been removed from MDN I'm removing the dev-doc-needed keyword, but feel free to add it back if I've missed anything