Closed Bug 620283 Opened 9 years ago Closed 9 years ago

crash [@ nsHTMLEditor::RemoveStyleInside] if !aAttribute

Categories

(Core :: DOM: Editor, defect, critical)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: timeless, Assigned: kaze)

References

(Blocks 1 open bug)

Details

(Keywords: coverity, crash, Whiteboard: [post-2.0])

Crash Data

Attachments

(1 file)

649 nsresult nsHTMLEditor::RemoveStyleInside(nsIDOMNode *aNode, 

677     // if we weren't passed an attribute, then we want to 
678     // remove any matching inlinestyles entirely
679     if (!aAttribute || aAttribute->IsEmpty())

764   if ( aProperty == nsEditProperty::font &&    // or node is big or small and we are setting font size
765        (nsHTMLEditUtils::IsBig(aNode) || nsHTMLEditUtils::IsSmall(aNode)) &&
766        aAttribute->LowerCaseEqualsLiteral("size"))
Assignee: nobody → ehsan
Whiteboard: [post-2.0]
Crash Signature: [@ nsHTMLEditor::RemoveStyleInside]
Assignee: ehsan → kaze
Status: NEW → ASSIGNED
Attached patch patch proposalSplinter Review
Attachment #548591 - Flags: review?(ehsan)
Attachment #548591 - Flags: review?(ehsan) → review+
http://hg.mozilla.org/mozilla-central/rev/f6545ec92d4c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.