182215 - Possible quoting problem

Status: RESOLVED DUPLICATE of bug 43267

(Core :: DOM: HTML Parser, defect)

Description

[Jeremy Sanders]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2) Gecko/20021126
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2) Gecko/20021126

The above page has a menu with dark green background and white text. With
Mozilla 1.2 the  menu has not got a green background any more (though it does
with Mozilla 1.1).

The html responsible is

(this was downloaded with wget)

    <td class="menu" width=102 style="background-color:"#004A25"; cursor:hand"
onmouseover="movein(this,'Home')" onmouseout="moveout(this)" height="26"> 
      <div align="center"><a href="/default.asp">Home</a></div>
    </td>

I suspect the quotes around #004a25 is a bug in the web page. Is this correct?

Oddly, if I do view source in mozilla 1.2, the code looks like
    <td class="menu" width=102 style="background-color:"#004A25; cursor:hand
onmouseover="movein(this,'Home')" onmouseout="moveout(this)" height="26"> 
      <div align="center"><a href="/default.asp">Home</a></div>

So the ending double quote gets removed by mozilla somewhere between downloading
it and viewing the source.

If you save the web page (in complete format), the text becomes
    <td class="menu" width="102" style="" #004a25="" ;="" cursor:hand=""
onmouseover="movein(this,'Products')" onmouseout="moveout(this)" height="26"> 

Very strange!!



Reproducible: Always

Steps to Reproduce:
1. Download the page
2. View source
3.

Actual Results:  
Mozilla:
o Doesn't show the green background (though this could be a web page bug)
o Removes a closing quote in "View Source"

Comment 1

[Boris Zbarsky [:bzbarsky]]

> I suspect the quotes around #004a25 is a bug in the web page

Yes, it is.

> Oddly, if I do view source 

View source uses the normal HTML tokenizer so it can do syntax highlighting, and
the token stream is messed up in this case.  There are bugs on this already.

> If you save the web page (in complete format)

This saves the page "as you see it" by design, so the behavior is correct.  The
CSS parses to nothing because the CSS parser can't make sense of it due to it
being truncated by the quote.

Over to parser for the tokenizer issue, but I doubt there's anything "fixable"
here... the tokenizer change was likely made to fix other problems with other
malformed content.

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

There's nothing we can do about the style being closed "prematurely". The
view-source issue is bug 43267, marking as a DUPE.

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