Closed Bug 903785 Opened 9 years ago Closed 1 year ago

ASSERTION: expected a TextNodeCorrespondenceProperty on nsTextFrame used for SVG text

Categories

(Core :: SVG, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: MatsPalmgren_bugz, Assigned: heycam)

References

Details

(Keywords: assertion, testcase)

content/svg/content/src/crashtests/499879-1.svg asserts:

###!!! ASSERTION: expected a TextNodeCorrespondenceProperty on nsTextFrame used for SVG text: 'Not Reached', file e:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/layout/svg/nsSVGTextFrame2.cpp, line 1273


https://tbpl.mozilla.org/php/getParsedLog.php?id=26401864&full=1&branch=mozilla-inbound

Windows 7 32-bit mozilla-inbound debug test crashtest on 2013-08-10 09:33:28 PDT for push 9813b854b713

slave: t-w732-ix-048

nsSVGTextFrame2::DoGlyphPositioning() [layout/svg/nsSVGTextFrame2.cpp:4792]
nsSVGTextFrame2::ReflowSVG() [layout/svg/nsSVGTextFrame2.cpp:3637]
nsSVGDisplayContainerFrame::ReflowSVG() [layout/svg/nsSVGContainerFrame.cpp:336]
nsSVGOuterSVGFrame::Reflow(nsPresContext *,nsHTMLReflowMetrics &,nsHTMLReflowState const &,unsigned int &) [layout/svg/nsSVGOuterSVGFrame.cpp:478]
nsContainerFrame::ReflowChild(nsIFrame *,nsPresContext *,nsHTMLReflowMetrics &,nsHTMLReflowState const &,int,int,unsigned int,unsigned int &,nsOverflowContinuationTracker *) [layout/generic/nsContainerFrame.cpp:971]
nsCanvasFrame::Reflow(nsPresContext *,nsHTMLReflowMetrics &,nsHTMLReflowState const &,unsigned int &) [layout/generic/nsCanvasFrame.cpp:498]
nsContainerFrame::ReflowChild(nsIFrame *,nsPresContext *,nsHTMLReflowMetrics &,nsHTMLReflowState const &,int,int,unsigned int,unsigned int &,nsOverflowContinuationTracker *) [layout/generic/nsContainerFrame.cpp:971]
nsHTMLScrollFrame::ReflowScrolledFrame(ScrollReflowState *,bool,bool,nsHTMLReflowMetrics *,bool) [layout/generic/nsGfxScrollFrame.cpp:450]
nsHTMLScrollFrame::ReflowContents(ScrollReflowState *,nsHTMLReflowMetrics const &) [layout/generic/nsGfxScrollFrame.cpp:613]
nsHTMLScrollFrame::Reflow(nsPresContext *,nsHTMLReflowMetrics &,nsHTMLReflowState const &,unsigned int &) [layout/generic/nsGfxScrollFrame.cpp:783]
nsContainerFrame::ReflowChild(nsIFrame *,nsPresContext *,nsHTMLReflowMetrics &,nsHTMLReflowState const &,int,int,unsigned int,unsigned int &,nsOverflowContinuationTracker *) [layout/generic/nsContainerFrame.cpp:971]
ViewportFrame::Reflow(nsPresContext *,nsHTMLReflowMetrics &,nsHTMLReflowState const &,unsigned int &) [layout/generic/nsViewportFrame.cpp:226]
PresShell::ProcessReflowCommands(bool) [layout/base/nsPresShell.cpp:7991]
PresShell::FlushPendingNotifications(mozilla::ChangesToFlush) [layout/base/nsPresShell.cpp:3890]
nsRefreshDriver::Tick(__int64,mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:1193]
The test is annotated "asserts(0-5)" in crashtests.list so I guess it's
a known problem, but there is no bug for it afaict. (bug 842909 is resolved fixed)
The uncaught JS exception from 499879-1.svg is the correct behaviour, since we're passing a bad value to SVGNumberlist.initialize() in the script.  The TextNodeCorrespondenceProperty assertion can't be from this test, since there's no SVG text around in this document.  Might it be coming from a previous test in the run?

(The "asserts(0-5)" is on 412104-1.svg, not 499879-1.svg, btw.)
The assertion that looks like it's in 499879-1.svg might be from 412104-1.svg.  (Maybe that's what you were suggesting in the first place, Mats.)
Yes, the test that asserts is 412104-1.svg.  Sorry, the stack dump for the last
assertion was intermingled with other test output in the log so I copy-pasted
the wrong line.  I'm pretty sure all those assertions belong to 412104-1.svg.
Assigning just to get this back in my todo list.
Assignee: nobody → cam
See Also: → 1429191
This is the same underlying cause as bug 1421807, for which I just landed a wallpaper for.
Depends on: 1421807
Depends on: 1469769

Closing this since bug 1421807 avoids the assertion.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.