130654 - Strict mode comment parsing of a single - is bogus?

Status: VERIFIED FIXED

(Core :: DOM: HTML Parser, defect)

Description

[Graham]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:0.9.9) Gecko/20020311
BuildID:    2002031104

Apart from the heading and left margin, the url
http://www.vm.ibm.com/pubs/ctlblk.html displays nothing where the "useful" text
should be. This fails on NT and OS/2 and did so for 0.9.8 too (and probably
earlier ones: I don't remember it ever working).

The URL works with NS4.79 (NT), NS4.61 (OS/2) and IE5.5 (NT).


Reproducible: Always
Steps to Reproduce:
1. Enter http://www.vm.ibm.com/pubs/ctlblk.html



Actual Results:  Header, left margin and a large white space where text should be.

Expected Results:  Header, left margin and text (mainly links).

Apologises for not better categorising this, I think it could fit under several
categories.

Comment 1

[Mats Palmgren (inactive)]

It's a comment parsing issue in strict mode. In the middle of the page is a
comment that can be reduced to:
<!-- --  - you are in a cell 600 wide -->
<!-- -- -->
TEXT

TEXT is not shown. Removing the DOCTYPE or the single '-' makes it visible.
My interpretation of the parsing of the above is, with my comments in []:

<![mdo]--[comment start] --[comment end]  - you are in a cell 600 wide
--[comment start]>
<!--[comment end] --[comment start] --[comment end]>[mdc, start content]
TEXT

Especially, I don't see why the single '-' should make any difference.

Comment 2

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 3

[harishd]

Heikki has rewritten comment parsing. Hopefully, his patch would fix this problem.

Giving bug to heikki.

Comment 4

[Heikki Toivonen (remove -bugzilla when emailing directly)]

This will be fixed by bug 110544.

Comment 5

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Fixed by bug 110544.

Comment 6

[Moied]

Verified fixed with build id 20020322 on WINXP