Bug 167717 - konqueror fails to load css
Summary: konqueror fails to load css
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-30 11:08 UTC by Jos van den Oever
Modified: 2008-07-30 19:56 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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!