Closed Bug 803562 Opened 12 years ago Closed 9 years ago

"ASSERTION: reflow roots should be reflowed at existing size..."

Categories

(Core :: SVG, defect)

x86_64
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed
b2g-v2.5 --- fixed

People

(Reporter: jruderman, Assigned: longsonr)

References

Details

(Keywords: assertion, testcase)

Attachments

(3 files)

Attached image testcase (deleted) —
###!!! ASSERTION: reflow roots should be reflowed at existing size and svg.css should ensure we have no padding/border/margin: 'aReflowState.ComputedWidth() == GetSize().width && aReflowState.ComputedHeight() == GetSize().height', file layout/svg/nsSVGForeignObjectFrame.cpp, line 158

###!!! ASSERTION: non-root frame's desired size changed during an incremental reflow: '(target == rootFrame && size.height == NS_UNCONSTRAINEDSIZE) || (desiredSize.width == size.width && desiredSize.height == size.height)', file layout/base/nsPresShell.cpp, line 7472
Attached file stacks (deleted) —
Still reproduces on trunk.
Attached patch here's one way to fix it (deleted) — Splinter Review
Attachment #8678606 - Flags: review?(dbaron)
Comment on attachment 8678606 [details] [diff] [review]
here's one way to fix it

Yeah, if we have border-width: 0 ! important, we should probably also have this.

Thanks.

r=dbaron
Attachment #8678606 - Flags: review?(dbaron) → review+
Assignee: nobody → longsonr
Flags: in-testsuite+
https://hg.mozilla.org/mozilla-central/rev/8622f5d07f93
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: