147446 - [FIX] A map definition inside a table and outside the tr element will add an extra space to the table

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect)

Description

[Roger C. Soares]

A table will have an extra space if a map is defined inside the table (see URL -
the first frame).

For other elements, the element is moved to outside the table and the table is
rendered fine.

Comment 1

[Roger C. Soares]

Attachment: Testcase showing a map inside a table and a br inside a table

The map element will cause an extra space in the table, the br element will be
move to before the table.

Comment 2

[Boris Zbarsky [:bzbarsky]]

To parser.

Comment 3

[/\/\arcio Galli]

Harish, 

We raised this one from the Evangelism Latin America effort. The site is part of
Major AOL partner in Brazil (Itau Bank). 

Comment 4

[/\/\arcio Galli]

I see. If I use the <map> element the generated DOM is: 
-------------------------------------------->
<table bgcolor="green">
<map></map>
<tbody>...

When I replace to another element (For example a div element), I have:
-------------------------->
<div></div>
<table bgcolor="green">
<tbody>...

Harish, 

Do you think we can consider the MAP like the other elements?

Comment 5

[Christopher Hoess (gone)]

The URL has been redesigned to eliminate the problem.

Comment 6

[Mats Palmgren (inactive)]

Attachment: Testcase #2

Comment 7

[Mats Palmgren (inactive)]

Taking this since I have a simple fix for it...

Comment 8

[Mats Palmgren (inactive)]

Attachment: Patch rev. 1

Remove MAP as valid child of TABLE/THEAD/TFOOT

Comment 9

[Mats Palmgren (inactive)]

The patch makes us compatible with NS4 and IE5.5/IE6.  This is also the right
thing to do with respect to HTML 4 spec.  Opera7 have *exactly* the same bug
as Mozilla currently has though.  I have tested this extensively; the regression
tests pass, the top100 list pass (which means none of them depend on this bug).
I have checked all Parser bugs that mentions "map" and "table" in the same bug
and found no regressions (among them I found bug 58942 which fixed the same bug
for TR).  I also checked all Layout bugs matching "<map" and "table" and saw
no problems.

Comment 10

[harishd]

Comment on attachment 126246 [details] [diff] [review]
Patch rev. 1

r=harishd

Comment 11

[Bernd]

Mats what is the status of this patch? Seems like it has a review :-)

Comment 12

[Stephen Donner [:stephend] Not actively reading bugmail]

Mats, is there still interest in this patch?

Comment 13

[Mats Palmgren (inactive)]

Checked in 2005-01-25 16:40 PDT.

-> FIXED