[Webkit-unassigned] [Bug 74864] There is additional space not belonged to a table between the table cells

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 6 03:40:08 PDT 2012


--- Comment #12 from Allan Sandfeld Jensen <allan.jensen at nokia.com>  2012-08-06 03:40:05 PST ---
> (In reply to comment #8)
> Also, I was looking at perhaps raising another bug which removes the following FIXME from the code (in RenderTableSection::spannedRows()):
>  // FIXME: Upper_bound might not be the correct algorithm here since it might skip empty rows, but it is
>     // consistent with behavior in the former point based hit-testing (but inconsistent with spannedColumns).
> I could add testcases with empty <TR>s to verify upper_bound() algorithm's behavior.

Make a test-case verifying or invalidating the problem would be great.

For this patch though, a guess the comment actually needs to be updated to remove the claim that it is inconsistant with spannedColumns, since you changed that.

But talking about what happens with empty-cells made me take another look at your empty-cell test-case. Since 'empty-cells' default to 'show' which means td3 and td4 renders their own border, doesn't your test-case actually demonstrating a bug when it says elementAtPoint on the borders of td3 hits td5? And a really big bug when hitting the td3 border from td4 suddenly hits td2?

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list