6749 - CDATA blocks: end tags and <!--

Status: VERIFIED FIXED

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

Description

[Hixie (not reading bugmail)]

The parser is not treating STYLE blocks as CDATA.

So, for example, the following:

   <STYLE type="text/x-dwim-styles">
       make arrow heads look like '<!--'.
   </STYLE>

...will turn the rest of the document into a comment, when it should not.

See the test page:
   http://www.bath.ac.uk/%7Epy8ieh/internet/projects/mozilla/viewsource.html
...which demonstrates these problems in terms of what happens in View Source.

(while we're at it, note that View Source renders the </SCRIPT> end tag as
simply </>. This is probably not intentional...)

Comment 1

[rickg]

Updated parser.

Comment 2

[Hixie (not reading bugmail)]

There are still some errors on this page.

  1. The </style> and </script> tags are not respecting the original document's
     case (they are always lower case).

  2. The comment <!-- in the third block is hiding the rest of the document!

Because of error 2 above, I am increasing the severity of this bug.

Comment 3

[harishd]

Assigning to myself

Comment 4

[harishd]

Parser does treat <STYLE> blocks as CDATA.

It's the non-ending comment inside the <SCRIPT> block that causes the
document to be hidden.

Comment 5

[Hixie (not reading bugmail)]

In that case, the parser is incorrectly treating CDATA. CDATA blocks should
not be parsed (otherwise they would be called PCDATA blocks). The only thing
that should be recognised in CDATA blocks is ETAGO, i.e.: "</". What you are
calling a 'non-ending comment' should not be treated as a comment at all.

See the first comment in this bug for an example of what I mean.

Comment 6

[harishd]

Since this bug has two parts, opening a new bug for the second problem.

Comment 7

[harishd]

Made </STYLE> and </SCRIPT>, etc. to represent document's case.
This bug can be closed.  I have opened a new bug (#7853)for the second problem
mentioned.

Marking the bug fixed.

Comment 8

[Hixie (not reading bugmail)]

Verified fixed in viewer 1999-06-14 build.
Verified that the main problem is now covered by bug 7853.