1440039 - Remove nsIDOMCanvasRenderingContext2D

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement)

Description

[Adrian Wielgosik [:adrian17]]

It's unimplemented (since bug 800556), has only constants, the only JS user looks like dead code:

https://searchfox.org/mozilla-central/source/browser/modules/WindowsPreviewPerTab.jsm#346

Comment 1

[Adrian Wielgosik [:adrian17]]

Attachment: Bug 1440039 - Remove nsIDOMCanvasRenderingContext2D.

Review commit: https://reviewboard.mozilla.org/r/222058/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/222058/

Comment 2

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Comment on attachment 8952829 [details]
Bug 1440039 - Remove nsIDOMCanvasRenderingContext2D.

https://reviewboard.mozilla.org/r/222058/#review227972

Thanks!

Comment 3

[Adrian Wielgosik [:adrian17]]

For the record, looks like the file with dead code is copied in comm-central.

https://searchfox.org/comm-central/source/suite/modules/WindowsPreviewPerTab.jsm#328

Comment 4

[Jorg K (CEST = GMT+2)]

Thanks for the heads-up. comm-central is home to code for Thunderbird and SeaMonkey, the suite/ directory is only for SeaMonkey. Redirecting the NI request to the SeaMonkey team.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b2e9d1ed15f4
Remove nsIDOMCanvasRenderingContext2D. r=qdot

Comment 6

[Dorel Luca [:dluca]]

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

Comment 7

[Frank-Rainer Grahl (:frg)]

Thanks for the heads-up. Will see that I get it fixed for suite.