Closed Bug 1395442 Opened 2 years ago Closed 2 years ago

Remove nsHTMLDocument::mFormControls because it's never set

Categories

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

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: ehsan, Assigned: ehsan)

Details

Attachments

(1 file)

This patch also removes some useless null checks becuase operator new never returns null.
Comment on attachment 8903034 [details] [diff] [review]
Remove nsHTMLDocument::mFormControls because it's never set

Didn't I review this in some other bug. Maybe it was something very similar.
Attachment #8903034 - Flags: review?(bugs) → review+
It was similar, you're thinking of bug 1392891.  I didn't realize at the time that this can be removed altogether.
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1116ed0b6942
Remove nsHTMLDocument::mFormControls because it's never set; r=smaug
Assignee: nobody → ehsan
https://hg.mozilla.org/mozilla-central/rev/1116ed0b6942
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.