DESCRIPTION: The mode detection for an HTML document that has an XML declaration (a pseudo-PI) is confused. It puts the parser into standard mode but somehow layout stays in quirks mode: http://www.bath.ac.uk/%7Epy8ieh/cgi/compat-test.pl?DOCTYPE=%3C%3Fxml+version%3D%221.0%22%3F%3E&MODE=full The tests should not contradict each other -- they should show the same result as http://www.bath.ac.uk/%7Epy8ieh/cgi/compat-test.pl?DOCTYPE=%3C%21DOCTYPE+HTML+PUBLIC+%22-%2F%2FW3C%2F%2FDTD+XHTML+1.0+Strict%2F%2FEN%22%3E&MODE=full

Note that this applies only to documents sent as text/html, not to ones sent as text/xml.

I really don't know what's preventing the parser from propagating the mode to layout, but I'm worried that it has something to do with "treating it as XML".

--> 0.9.5

--> 0.9.6

Out of time :-( Moving to 0.9.7

WFM if I add |<!DOCTYPE >|. AIUI layout gets the mode in HTMLContentSink::AddDocTypeDecl() and stays in quirks mode if the content sink gets no doctype node.

The mode should be determined in nsParser::DetermineParseMode. (I wanted to unify that with the doctype token handling, but the parser was just too complicated.)

Woo hoo! A fix!

Comment on attachment 59405 [details] [diff] [review] set layout mode whether a DOCTYPE exists or not sr=jst

Comment on attachment 59405 [details] [diff] [review] set layout mode whether a DOCTYPE exists or not r=heikki

FIXED.

verified fixed with build ID 20020117 on win2k and fix checked in cvs (rev 3.502)