Closed Bug 1208884 Opened 4 years ago Closed 4 years ago

Fix a null check ordering bug in nsHTMLEditRules::SplitParagraph

Categories

(Core :: Editor, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: Ehsan, Unassigned)

References

Details

Attachments

(1 file, 1 obsolete file)

Found by Viva64.
Found by Viva64.
Attachment #8666526 - Flags: review?(roc)
Attachment #8666500 - Attachment is obsolete: true
I don't think this solution makes sense. In many cases the line directly about this is an NS_ENSURE_STATE(mHtmlEditor), and there are cases where that exists multiple times.
https://hg.mozilla.org/mozilla-central/rev/25ddce309321
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
(In reply to Josh Matthews [:jdm] from comment #4)
> I don't think this solution makes sense. In many cases the line directly
> about this is an NS_ENSURE_STATE(mHtmlEditor), and there are cases where
> that exists multiple times.

You're right!  Filed bug 1209037.
Blocks: 710966
You need to log in before you can comment on or make changes to this bug.