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

Status: RESOLVED FIXED

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

Description

[Cameron McCormack (:heycam)]

###!!! 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.

Comment 1

[Cameron McCormack (:heycam)]

This is fixed in the latest mozilla-central -> stylo merge.

Comment 2

[Cameron McCormack (:heycam)]

This didn't get fixed after all (see bug 1334938); sorry for the noise.

Comment 3

[Hiroyuki Ikezoe (:hiro)]

Now we get another assertion.
ASSERTION: intrinsic widths out of order: 'aMinCoord <= aPrefCoord', file /home/ikezoe/stylo/layout/tables/nsTableColFrame.h, line 166

Comment 4

[Xidorn Quan [:xidorn] UTC+10]

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 5

[Xidorn Quan [:xidorn] UTC+10]

Attachment: Bug 1324682 - Remove annotation of this bug.

Review commit: https://reviewboard.mozilla.org/r/178216/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/178216/

Comment 6

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8906465 [details]
Bug 1324682 - Remove annotation of this bug.

https://reviewboard.mozilla.org/r/178216/#review183112

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1cc09280f38c
Remove annotation of this bug. r=xidorn

Comment 8

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/1cc09280f38c