Assertion failure: !nsSVGUtils::OuterSVGIsCallingReflowSVG(this) (do not call ScheduleReflowSVGNonDisplayText when the outer SVG frame is under ReflowSVG)

RESOLVED FIXED in Firefox 68

Status

()

defect
P3
normal
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: violet.bugreport, Assigned: heycam)

Tracking

({assertion, regression, testcase})

Trunk
mozilla68
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox-esr60 wontfix, firefox66 wontfix, firefox67 wontfix, firefox68 fixed)

Details

Attachments

(2 attachments)

Reporter

Description

3 months ago
Posted file testcase.html

Bug 897887 has a regression somewhere. It was supposed to have been fixed 6 years ago, but now the testcase in that bug still triggers the same assertion failure on the trunk.

Reporter

Updated

3 months ago
Blocks: 897887
Assignee

Comment 1

3 months ago

My fault for not adding the test case when landing bug 897887.

Assignee

Updated

3 months ago
Assignee: nobody → cam
Status: NEW → ASSIGNED
Assignee

Updated

3 months ago
Priority: -- → P3
Reporter

Comment 4

3 months ago

Thanks for fixing this bug. Could you also review my patch to bug 1072758? I actually discovered this regression when working on bug 1072758. I want to be confirmed there is no caveat in my patch.

Flags: needinfo?(cam)
Assignee

Comment 5

3 months ago

Sure, I will comment over on that review. Thanks for picking up that assertion fix!

Flags: needinfo?(cam)

Comment 6

3 months ago
Pushed by cmccormack@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7517b9beb5e6
Don't schedule SVG text reflow in response to bidi resolution. r=jwatt

Comment 7

3 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.