42268 - Browser displays HTML comments on the rendered page

Status: VERIFIED DUPLICATE of bug 42270

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

Description

[Stephan Niemz]

CVS checkout at Mon Jun 12 21:06:12 UTC 2000

    The browser displays Javascript code in http://bugzilla.mozilla.org/ (and
perhaps others) on the page itself, and renders the page completely wrong.

Comment 1

[Stephan Niemz]

Attachment: Sample screenshot showing this bug

Comment 2

[Matthew Cline]

I can't seem to duplicate this with the Bugzilla home page, but I can with
http://www.zdnet.com/eweek/stories/general/0,11011,2585114,00.html


Build 2000061214, Linux 2.2.14 i686, RedHat 6.1

Comment 3

[Blake Ross]

sounds like 42270 to me

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

Comment 4

[Stephan Niemz]

    Reopening, because bug 42270 is marked resolved/fixed, but I still get
this bug. Currently I'm seeing two incarnations:

1.
Start Mozilla and load http://bugzilla.mozilla.org/
At the top of the page I get:

The contents of this file are subject to the Mozilla Public License Version
1.1 (the "license"); you may not use this file except in compliance with
the license.

and so on, which is part of an HTML comment and shouldn't be displayed.
When loading this page a second time (e. g. by pressing the reload button),
the comment disappears.

2.
I'm going to attach an HTML file which should be rendered as an empty page
because it only contains comments in the body. Instead you see:

	form action="nothinguseful" method="post"

and so on. This time it doesn't disappear after reloading.

    My last CVS checkout was on 13.06.2000, 11:58 UTC. I have built a
non-debug and optimized Mozilla in an obj-* subdirectory, if that matters.

Comment 5

[Stephan Niemz]

Attachment: Sample HTML file showing this bug

Comment 6

[karnaze (gone)]

Pierre, please triage these from Clayton's list.

Comment 7

[Hixie (not reading bugmail)]

This is Rick's -- parser problem. Confirmed XP, I've seen this on Win32.

Comment 8

[Pierre Saslawsky]

It could be a dup of bug 42340 except that it can be reproduced on all platforms.

Reassigned to rickg, CCd harishd.

Comment 9

[Stephan Niemz]

    Yes, the description of bug 42340 is exactly what I've seen here and
described above - only on Linux, not on the Mac.

    Maybe the second part (the attached HTML file) is not the same bug, since
you don't have to restart the browser to get it again. I don't know. The
interesting thing is that if you remove a few lines of x-es in that file,
the bug disappears. It seams to depend on the huge amount of content (comment)
before that other comment which then gets displayed.

Comment 10

[harishd]

The problem was due to lack of error propagation [ refer to the patch in 
bug 42340 ], in CComment::Consume().

Comment 11

[rickg]

Fixed in my tree.

Comment 12

[rickg]

The bugzilla problem was related to a bug in the StrictDTD. This bughas to do 
with a bug in the parser. The bugzilla issue was resolved (42268) is fixed in my 
tree awaiting a chance to checkin.

Comment 13

[rickg]

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

Comment 14

[Stephan Niemz]

    I have tested it (after another checkout) with and without that patch
in bug 42270, and it fixes both issues for me. Thanks!

Comment 15

[Stephan Niemz]

Sorry, I meant the patch attached to bug 42340. It's here:

http://bugzilla.mozilla.org/showattachment.cgi?attach_id=10156

Comment 16

[Scott Brodmerkle]

Verified dupe.