"ASSERTION: vertical damage extends outside table" with changing colspan

RESOLVED FIXED in mozilla12

Status

()

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

People

(Reporter: Jesse Ruderman, Assigned: Bernd)

Tracking

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

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

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

6 years ago
Created attachment 581129 [details]
testcase

###!!! ASSERTION: vertical damage extends outside table: '(aValue).YMost() <= GetRowCount()', file layout/tables/nsTableFrame.cpp, line 3810
(Reporter)

Comment 1

6 years ago
Created attachment 581130 [details]
stack trace
(Assignee)

Comment 2

6 years ago
Created attachment 584450 [details] [diff] [review]
patch
Assignee: nobody → bernd_mozilla
Status: NEW → ASSIGNED
(Assignee)

Comment 3

6 years ago
https://tbpl.mozilla.org/?tree=Try&rev=a240e292b4da
(Assignee)

Comment 4

6 years ago
review hint
http://mxr.mozilla.org/mozilla-central/source/layout/tables/nsCellMap.cpp#2150

we need both the colspan and the rowspan as effective (aka in real cellmap coordinates)

this code is from http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/tables/nsCellMap.cpp&rev=3.44
Bernd, did you intend to ask me to review this patch?
(Assignee)

Comment 6

6 years ago
Comment on attachment 584450 [details] [diff] [review]
patch

Yes, the problem is old but currently its the only remainder of the damagearea patch.
Attachment #584450 - Flags: review?(matspal)

Updated

6 years ago
Attachment #584450 - Flags: review?(matspal) → review+
(Assignee)

Comment 7

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/92efb7a05df5
(Assignee)

Updated

6 years ago
Flags: in-testsuite+
https://hg.mozilla.org/mozilla-central/rev/92efb7a05df5
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.