[BC] ASSERTION: CellIterator program error: 'PR_FALSE'

RESOLVED FIXED

Status

()

Core
Layout: Tables
RESOLVED FIXED
12 years ago
6 years ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

(Blocks: 2 bugs, {assertion, testcase})

Trunk
assertion, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

12 years ago
To reproduce, load the testcase in a debug build.

###!!! ASSERTION: CellIterator program error: 'PR_FALSE', file /Users/admin/trunk/mozilla/layout/tables/nsTableFrame.cpp, line 7148

I think this is related to bug 315549.
(Reporter)

Comment 1

12 years ago
Created attachment 208631 [details]
testcase
OS: Mac OS X 10.2 → All
Hardware: Macintosh → All
Created attachment 234590 [details] [diff] [review]
wip

I know close to nothing about this code so this is just a stab in the dark...

Maybe we shouldn't require a cell frame unless we are painting for a cell?
This patch fixes the assertion.
Created attachment 234591 [details]
Screenshot (with "wip" applied)

Note the missing pixel on bottom border.

Comment 4

11 years ago
PLEASE, no work on BC other than making the cells paint its border by themself. Its waisted time. The code should be removed and not patched.
(Reporter)

Comment 5

11 years ago
Created attachment 244057 [details]
another testcase
(Reporter)

Updated

11 years ago
Blocks: 339128
(Reporter)

Updated

11 years ago
Summary: ASSERTION: CellIterator program error: 'PR_FALSE' → [BC] ASSERTION: CellIterator program error: 'PR_FALSE'
(Reporter)

Comment 6

10 years ago
WFM with "another testcase", but still happens for "testcase".

Bernd, can you do me a favor and mark this bug as depending on the bug you alluded to in comment 4?
(Reporter)

Comment 7

7 years ago
Now "another testcase" is still WFM, and "testcase" gives me this assertion instead of the CellIterator one in the bug summary:

###!!! ASSERTION: invalid BC damage area: 'PR_FALSE', file /Users/jruderman/central/layout/tables/nsTableFrame.cpp, line 3760

Comment 8

6 years ago
fixed by bug 460637

https://hg.mozilla.org/integration/mozilla-inbound/rev/2a15404c1d71
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.