"ASSERTION: we're not in galley mode" with table, -moz-column

RESOLVED FIXED

Status

()

RESOLVED FIXED
11 years ago
9 years ago

People

(Reporter: jruderman, Assigned: bernd_mozilla)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
x86
Mac OS X
assertion, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

11 years ago
Created attachment 317137 [details]
testcase

###!!! ASSERTION: we're not in galley mode: 'NS_UNCONSTRAINEDSIZE == aReflowState.reflowState.availableHeight', file /Users/jruderman/trunk/mozilla/layout/tables/nsTableRowGroupFrame.cpp, line 938

Seems to be quirks-mode-only, which makes it hard to create a mostly-markup testcase (since the HTML parser doesn't like text in tables or tables without tbody elements).
(Assignee)

Comment 1

11 years ago
the assert is obsolete now as is the comment above the function 
Assignee: nobody → bernd_mozilla
(Assignee)

Comment 2

10 years ago
Created attachment 334149 [details] [diff] [review]
patch
Attachment #334149 - Flags: superreview?(bzbarsky)
Attachment #334149 - Flags: review?(bzbarsky)
Attachment #334149 - Flags: superreview?(bzbarsky)
Attachment #334149 - Flags: superreview+
Attachment #334149 - Flags: review?(bzbarsky)
Attachment #334149 - Flags: review+
(Assignee)

Comment 3

10 years ago
fix checked in
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
I checked the testcase as a crashtest.  It triggered an edge case
in the code so I thought it would be good to have in the testsuite.
(It passed all platforms on MozillaTry unit test)
http://hg.mozilla.org/mozilla-central/rev/62124009b06f
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.