Closed
Bug 384391
Opened 18 years ago
Closed 18 years ago
<svg:circle> as root element causes "ASSERTION: null ptr"
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: longsonr)
References
Details
(Keywords: assertion, testcase)
Attachments
(4 files, 1 obsolete file)
341 bytes,
application/xhtml+xml
|
Details | |
1.12 KB,
patch
|
bzbarsky
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
4.74 KB,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
3.50 KB,
patch
|
Details | Diff | Splinter Review |
warning: trying to construct an SVGOuterSVGFrame for a content element that doesn't support the right interfaces
###!!! ASSERTION: null ptr: 'nsnull != aFrame', file /Users/jruderman/trunk/mozilla/layout/generic/nsFrameList.cpp, line 92
Attachment #268414 -
Flags: review?(bzbarsky)
Comment 2•18 years ago
|
||
tor, is that really the right patch?
Attachment #268414 -
Attachment is obsolete: true
Attachment #268421 -
Flags: review?(bzbarsky)
Attachment #268414 -
Flags: review?(bzbarsky)
Comment 4•18 years ago
|
||
Comment on attachment 268421 [details] [diff] [review]
right patch this time
Looks reasonable.
Attachment #268421 -
Flags: review?(bzbarsky) → review+
Attachment #268421 -
Flags: superreview?(roc)
Reporter | ||
Comment 5•18 years ago
|
||
Now this also triggers:
###!!! ASSERTION: Can't create frame! Content is not an SVG 'svg' element!: 'Error', file nsSVGOuterSVGFrame.cpp, line 140
Should I file a separate bug for that, or will the patch here fix it? (That assertion is new with bug 385218, btw.)
Assignee | ||
Comment 6•18 years ago
|
||
(In reply to comment #5)
This patch should prevent that assertion too.
Attachment #268421 -
Flags: superreview?(roc) → superreview+
Checked in.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Flags: in-testsuite?
Assignee | ||
Comment 8•18 years ago
|
||
The previous patch seems to have caused bug 385840 and bug 385852
Attachment #269843 -
Flags: review?(bzbarsky)
Assignee | ||
Comment 9•18 years ago
|
||
Assignee | ||
Updated•18 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 10•18 years ago
|
||
Comment on attachment 269843 [details] [diff] [review]
fix crashes
apparently bz is away.
Attachment #269843 -
Flags: review?(bzbarsky) → review?(roc)
Attachment #269843 -
Flags: superreview+
Attachment #269843 -
Flags: review?(roc)
Attachment #269843 -
Flags: review+
Assignee | ||
Comment 11•18 years ago
|
||
Comment on attachment 269843 [details] [diff] [review]
fix crashes
checked in.
Assignee | ||
Updated•18 years ago
|
Status: REOPENED → RESOLVED
Closed: 18 years ago → 18 years ago
Resolution: --- → FIXED
Updated•14 years ago
|
Assignee: nobody → longsonr
You need to log in
before you can comment on or make changes to this bug.
Description
•