Closed Bug 182215 Opened 22 years ago Closed 20 years ago

Possible quoting problem

Categories

(Core :: DOM: HTML Parser, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 43267

People

(Reporter: jss, Assigned: harishd)

References

()

Details

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"
> 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.
Component: DOM to Text Conversion → Parser
QA Contact: sujay → moied
Depends on: 188609
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 ***
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.