Bug 167717

Summary:	konqueror fails to load css
Product:	[Applications] konqueror	Reporter:	Jos van den Oever <jos>
Component:	khtml	Assignee:	Konqueror Developers <konq-bugs>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Jos van den Oever 2008-07-30 11:08:53 UTC

Version:            (using KDE 4.1.0)
Installed from:    Ubuntu Packages

On the website http://teamcity.jetbrains.com/guestLogin.html?guest=1 the page is rendered incorrectly.
It seems as if the css is not loaded at all.

Comment 1 Maksim Orlovich 2008-07-30 17:47:47 UTC

Looks like the parser can't recover from the badly malformed html properly and loses the <style> tag (and also the <title>). Amusing that they use an xhtml dtd here..

Comment 2 Maksim Orlovich 2008-07-30 18:03:17 UTC

SVN commit 839700 by orlovich:

Now that the HTML parser adds doctype nodes, it can no longer be sure that the
<html> will be the first child of #document, so use ->documentElement() instead.
Restores recovery from mis-structured head/body, w/o which we could even lose 
<style> or <title> elements in some cases of malformed input
BUG:166450
BUG:167717 


 M  +5 -5      htmlparser.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=839700

Comment 3 Maksim Orlovich 2008-07-30 18:15:01 UTC

SVN commit 839703 by orlovich:

Regression test for 167717/166450

CCBUG:167717
CCBUG:166450


 A             baseline/parser/167717.html-dom  
 M  +3 -1      baseline/parser/svnignore  
 A             tests/parser/167717.html  


WebSVN link: http://websvn.kde.org/?view=rev&revision=839703

Comment 4 Jos van den Oever 2008-07-30 19:56:46 UTC

Thanks for the quick fix. That is a beautiful unit test system!