15999 - {ib} {inc} Page not rendered correctly

Status: VERIFIED FIXED

(Core :: DOM: HTML Parser, defect, P3)

Description

[moshev]

The home page of largest Israely ISP (Netvision)
is not rendered correctly (hebrew font issue aside),
Will attach two screenshots, one with M10, and another NS 4.7
using mixed RH/Mandrake 6.0/6.1

Comment 1

[moshev]

Attachment: This one is with NS 4.7 and is ok

Comment 2

[moshev]

Attachment: This one is M10, and is wrong

Comment 3

[Nicholas Cull]

Attachment: Test case showing anchor containing block elements

Comment 4

[Nicholas Cull]

This page has a lot of really nasty HTML in it!

What this problem appears to boil down to is an anchor near the top of the page
that doesn't get closed before hitting a whole heap of nested tables. For some
reason this is preventing the images inside one of the tables from displaying,
although at least one of them does get loaded.

When you hit reload under Mozilla.exe, the left hand image will appear, but the
right hand image will stubbornly refuse.

This may be a manifestation of an already known residual style bug. Right now I'm
too tired to poke around Bugzilla to find any duplicates.

I put this test case together using the 1999101415 build of Mozilla running on a
Windows NT 4 Workstation.

Comment 5

[rickg]

Yup -- it's residual style. Putting on the radar.

Comment 6

[harishd]

In attachment #3 [details] [diff] [review] if the anchor is followed by a space or newline the page loads
fine!!! That is, replace <A ..><DIV..></DIV> with <A ..> <DIV ..></DIV>
(note the whitespace).  Is this residual style at all??

Comment 7

[harishd]

Kipp, the problem that I mentioned in my 10/20 comment could be related to
Layout.  I agree that anchor cannot contain block-level elements but that's a
different problem.

Comment 8

[kipp]

I've added the ib and inc markers to the issue with testcase 3 -- for some
reason when the table is inside the A element the image size update doesn't
trigger the proper reflow of the second table cell. If I remove the A element
that contains the table *all is well*. Something to do with incremental reflow
and ib situations.

Comment 9

[kipp]

I reworked the way that blocks inside of inlines are handled and move it out of
the layout logic and into the frame construction code. This also fixes 16737

Comment 10

[leger]

Setting QA Contact.

Comment 11

[buster]

mine now

Comment 12

[buster]

mistakenly removed the Fixed resolution.

Comment 13

[bsharma]

updated qa contact.

Comment 14

[bsharma]

Verified on:
Build: 2001-04-25-09-Trunk
Platform: WinNT

The above url and the test case loads fine.