Bug 68714 - Consider increasing the maximal rowspan value
Summary: Consider increasing the maximal rowspan value
Status: RESOLVED DUPLICATE of bug 171322
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
: 96849 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-09-23 12:23 PDT by Boris Zbarsky
Modified: 2022-08-20 15:31 PDT (History)
10 users (show)

See Also:


Attachments
Example of the issue (taken from downstream Chromium bug) (198.23 KB, text/html)
2012-10-05 17:32 PDT, Julien Chaffraix
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Boris Zbarsky 2011-09-23 12:23:25 PDT
This is currently set at 8190 because that's what Gecko had it set to.  We're probably increasing it to 65534 in Gecko because there are in fact use cases that want pretty large rowspans; see https://bugzilla.mozilla.org/show_bug.cgi?id=688405
Comment 1 Julien Chaffraix 2012-10-05 17:29:52 PDT
*** Bug 96849 has been marked as a duplicate of this bug. ***
Comment 2 Julien Chaffraix 2012-10-05 17:32:16 PDT
Created attachment 167423 [details]
Example of the issue (taken from downstream Chromium bug)

Copying relevant data from the duplicated bug:

FF currently uses 2^16 - 2 as the max value for rowspan. This value is chosen due a limitation in there impl of cell maps. 
http://mxr.mozilla.org/mozilla-central/source/layout/tables/celldata.h#17
Comment 3 Pravin D 2012-10-08 00:09:20 PDT
@Julien 
If we are planning to increase the rowspan limit, what are the relevant factors that needs to be considered ?
Comment 4 Ahmad Saleem 2022-08-12 09:21:40 PDT
From attached test case - all browsers (Chrome Canary 106, Firefox Nightly 105 and Safari Technical Preview 151) show result of 8500 rows.

Since all browsers are matching these days, I am going to mark this as "RESOLVED WONTFIX", if it is still an issue, please reopen it. Thanks!
Comment 5 Alexey Proskuryakov 2022-08-20 15:31:33 PDT
This is not the best test, as is only tests up to 8500. In fact, everyone is aligned at 65534 now.

One thing I noticed was this code still in WebKit:

Source/WebCore/mathml/MathMLElement.cpp:    static const unsigned maxRowspan = 8190; // This constant comes from HTMLTableCellElement.

I don't know if this is wrong or important, someone is encouraged to file a new bug if this suggests an actual problem.

*** This bug has been marked as a duplicate of bug 171322 ***