Closed Bug 1324682 Opened 3 years ago Closed 2 years ago

stylo: layout/tables/crashtests/420654-1.xhtml non-fatally asserts with "intrinsic widths out of order: 'aMinCoord <= aPrefCoord"

Categories

(Core :: CSS Parsing and Computation, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: heycam, Assigned: xidorn)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

###!!! ASSERTION: negative widths not allowed: 'aResult >= 0', file /home/worker/workspace/build/src/layout/base/nsLayoutUtils.cpp, line 4518

Has a very large width property value.
This is fixed in the latest mozilla-central -> stylo merge.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
This didn't get fixed after all (see bug 1334938); sorry for the noise.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Priority: -- → P3
Now we get another assertion.
ASSERTION: intrinsic widths out of order: 'aMinCoord <= aPrefCoord', file /home/ikezoe/stylo/layout/tables/nsTableColFrame.h, line 166
Summary: stylo: layout/tables/crashtests/420654-1.xhtml non-fatally asserts with "negative widths not allowed" → stylo: layout/tables/crashtests/420654-1.xhtml non-fatally asserts with "intrinsic widths out of order: 'aMinCoord <= aPrefCoord"
Priority: P3 → --
Priority: -- → P3
This doesn't seem to be a Stylo-specific issue at all.

This test was marked with a "asserts-if(stylo,4)" in bug 1324624, but this special marking is latter removed in bug 1342596 without the related bug being removed at the same time.

I think we can just remove the bug reference and close this bug.
Comment on attachment 8906465 [details]
Bug 1324682 - Remove annotation of this bug.

https://reviewboard.mozilla.org/r/178216/#review183112
Attachment #8906465 - Flags: review+
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1cc09280f38c
Remove annotation of this bug. r=xidorn
https://hg.mozilla.org/mozilla-central/rev/1cc09280f38c
Status: REOPENED → RESOLVED
Closed: 3 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Assignee: nobody → xidorn+moz
You need to log in before you can comment on or make changes to this bug.