Closed
Bug 620283
Opened 14 years ago
Closed 13 years ago
crash [@ nsHTMLEditor::RemoveStyleInside] if !aAttribute
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
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)
|
1.28 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
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"))
|
||
Updated•14 years ago
|
Assignee: nobody → ehsan
Whiteboard: [post-2.0]
|
||
Updated•13 years ago
|
Crash Signature: [@ nsHTMLEditor::RemoveStyleInside]
|
|
||
Updated•13 years ago
|
Assignee: ehsan → kaze
|
Assignee | |
Updated•13 years ago
|
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 1•13 years ago
|
||
Attachment #548591 -
Flags: review?(ehsan)
|
|
||
Updated•13 years ago
|
Attachment #548591 -
Flags: review?(ehsan) → review+
|
||
Comment 2•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/f6545ec92d4c
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
|
||
Updated•6 years ago
|
Blocks: coverity-analysis
You need to log in
before you can comment on or make changes to this bug.
Description
•