Closed Bug 1252839 Opened 4 years ago Closed 4 years ago

Remove some if stmt after allocation with 'new'.

Categories

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

47 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: baku, Assigned: baku)

Details

(Whiteboard: btpp-active)

Attachments

(2 files)

Attached patch docShell.patchSplinter Review
No description provided.
Attachment #8725646 - Flags: review?(bzbarsky)
Attached patch xpfe.patchSplinter Review
Attachment #8725658 - Flags: review?(bzbarsky)
Comment on attachment 8725646 [details] [diff] [review]
docShell.patch

r=me
Attachment #8725646 - Flags: review?(bzbarsky) → review+
Comment on attachment 8725658 [details] [diff] [review]
xpfe.patch

>+++ b/docshell/base/nsDocShell.cpp

Seem out of place in this patch.

r=me with that.
Attachment #8725658 - Flags: review?(bzbarsky) → review+
Whiteboard: btpp-active
https://hg.mozilla.org/mozilla-central/rev/307f62ff2fc9
https://hg.mozilla.org/mozilla-central/rev/32e49c1e3a06
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.