Closed Bug 1381584 Opened 3 years ago Closed 3 years ago

Remove cite field from nsIDOMHTMLQuoteElement

Categories

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

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

Details

Attachments

(1 file)

This is the only field, but it does not appear to be used from C++. There's an equivalent field on HTMLQuoteElement.webidl so I think this is okay to remove.
It doesn't actually matter whether there's an equivalent field on the webidl.  If it's not scriptable and not used from C++, it's OK to remove.
Comment on attachment 8887181 [details]
Bug 1381584 - Remove unused cite field from nsIDOMHTMLQuoteElement.

https://reviewboard.mozilla.org/r/157934/#review163070
Attachment #8887181 - Flags: review?(bzbarsky) → review+
Pushed by amccreight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5a3f91efbc00
Remove unused cite field from nsIDOMHTMLQuoteElement. r=bz
https://hg.mozilla.org/mozilla-central/rev/5a3f91efbc00
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.