11061 - {sink} [4.xP] Content in tables containing <STYLE> and <SCRIPT> tags doubles up

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect, P3)

Description

[David Hallowell]

Overview Description:
When visiting http://www.guardian.co.uk/ (link in URL field is a snapshot of the
site) the Guardian/Observer logos near the top of the page appear twice, where
in Netscape/IE they only appear once as intended.
Investigation revealed that this problem occurs in tables that contain the <td>
<style> and <script> tags, if either of these tags are removed the duplication
is stopped. This seems to apply to both text and images. I am finalising a
testcase which I'll attach shortly.

Steps to Reproduce:
1) Visit http://www.guardian.co.uk/ or any other of the Guardian's websites.
or use my testcase.

Actual Results:
The Guardian/Observer logo is shown twice one above the other.

Expected Results:
The Guardian/Observer logo should only be shown once. It is only called once in
the HTML code.

Build Date & Platform Bug Found:
Windows 98 apprunner build: 1999073108
This is what I've tested my testcase on.

Additional Builds and Platforms Tested On:
First noticed in release M8 (Windows and Linux) as this was the first time I
visited that particular site with Mozilla

Additional Information:
Mozilla ignores the comments if a </SCRIPT> tag or </STYLE> tag is commented out
(this only seems to affect the closing tags. I hinted on this in my comments on
bug #7590 for the </SCRIPT> tag and now investigations seem to make it apply to
the </STYLE> tag too. I'll make a testcase and file a separate bug for this
soon.

Comment 1

[David Hallowell]

Attachment: Testcase showing problem with tables containg SCRIPT and STYLE tags

Comment 2

[karnaze (gone)]

Harish, I'm not sure if the parser is to blame or the content sink, but this is
not a table bug. nsCSSFrameConstructor::ConstructTableFrame is getting called
twice which explains the two tables on the attachement (put border=5 on the
table).

Comment 3

[Peter Linss]

Taking this. This is a notification problem in the content sink. I'll be
re-working all that code in the next few weeks (after returning from CSS w/g).

Comment 4

[Peter Linss]

Moving all content sink issues to M14.

Comment 5

[Peter Linss]

Make that M14.

Comment 6

[Peter Linss]

*** Bug 16131 has been marked as a duplicate of this bug. ***

Comment 7

[Peter Linss]

Thanks for taking this Vidur. Many of these are likely dups. I haven't marked
them as such to make sure I had the testcases handy...

Comment 8

[Mats Palmgren (inactive)]

*** Bug 16227 has been marked as a duplicate of this bug. ***

Comment 9

[vidur (gone)]

Fixed by sink changes on 10/26/1999.

Comment 10

[Christine Hoffman]

Using 11/16 builds across platform, verified bug fixed

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8e5a2a80e1d9
[fenix] Refactors the TabTrayDialogFragment to ditch that weird interactor (https://github.com/mozilla-mobile/fenix/pull/11061)