Closed Bug 1311673 Opened 4 years ago Closed 4 years ago

ImageDocument::ImageResizingEnabled is unused


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

Not set



Tracking Status
firefox52 --- fixed


(Reporter: Gijs, Assigned: edgar)


(Keywords: addon-compat)


(1 file)

AFAICT it always returns true, but nobody actually calls either the XPIDL or webidl implementations anywhere, so we're just shipping dead stuff. I found exactly 1 add-on "using" this read-only prop: - but afaict that function gets called in exactly 1 circumstance, namely if the image width and height are both exactly 24 pixels. That seems like buggy code - I don't think that justifies keeping our dead bits around.
Edgar, this seems in your wheelhouse :)
Flags: needinfo?(echen)
Hi Olli, I am willing to take this bug to remove ImageDocument.imageResizingEnabled, but it might cause add-on compatibility problems, although there seems only one add-on using this prop. How do we usually deal with this situation?
Flags: needinfo?(echen) → needinfo?(bugs)
jorge could answer to that.

Is the addon which is using this updated recently? Is it compatible with current FF release?
Flags: needinfo?(bugs) → needinfo?(jorge)
I think you could also just contact the addon author and ask whether it is ok to remove the property.
It's a very old add-on, probably abandoned. I'll include this in the compat communication, but the impact is minimal, so please go ahead with it.
Flags: needinfo?(jorge)
Keywords: addon-compat
Thanks for the comments, Jorge and Olli.
Assignee: nobody → echen
Attached patch Patch, v1Splinter Review
Attachment #8807031 - Flags: review?(bugs)
Attachment #8807031 - Flags: review?(bugs) → review+
Keywords: checkin-needed
Pushed by
Remove ImageDocument::ImageResizingEnabled. r=smaug
Keywords: checkin-needed
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.