Last Comment Bug 323604 - [BC] ASSERTION: CellIterator program error: 'PR_FALSE'
: [BC] ASSERTION: CellIterator program error: 'PR_FALSE'
: assertion, testcase
Product: Core
Classification: Components
Component: Layout: Tables (show other bugs)
: Trunk
: All All
-- normal (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
Depends on: 315549
Blocks: stirdom stirtable
  Show dependency treegraph
Reported: 2006-01-16 01:22 PST by Jesse Ruderman
Modified: 2011-12-04 00:37 PST (History)
5 users (show)
bernd_mozilla: in‑testsuite+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

testcase (432 bytes, text/html)
2006-01-16 01:34 PST, Jesse Ruderman
no flags Details
wip (3.66 KB, patch)
2006-08-19 08:17 PDT, Mats Palmgren (:mats)
no flags Details | Diff | Splinter Review
Screenshot (with "wip" applied) (1.54 KB, image/png)
2006-08-19 08:20 PDT, Mats Palmgren (:mats)
no flags Details
another testcase (610 bytes, application/xhtml+xml)
2006-10-29 23:07 PST, Jesse Ruderman
no flags Details

Description User image Jesse Ruderman 2006-01-16 01:22:36 PST
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.
Comment 1 User image Jesse Ruderman 2006-01-16 01:34:44 PST
Created attachment 208631 [details]
Comment 2 User image Mats Palmgren (:mats) 2006-08-19 08:17:53 PDT
Created attachment 234590 [details] [diff] [review]

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.
Comment 3 User image Mats Palmgren (:mats) 2006-08-19 08:20:14 PDT
Created attachment 234591 [details]
Screenshot (with "wip" applied)

Note the missing pixel on bottom border.
Comment 4 User image Bernd 2006-08-20 10:42:39 PDT
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.
Comment 5 User image Jesse Ruderman 2006-10-29 23:07:22 PST
Created attachment 244057 [details]
another testcase
Comment 6 User image Jesse Ruderman 2007-07-06 02:30:22 PDT
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?
Comment 7 User image Jesse Ruderman 2010-06-02 19:49:04 PDT
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

Note You need to log in before you can comment on or make changes to this bug.