Summary: | konqueror isn't able to handle ubs.com: "Loading..." | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Janet <bugzilla> |
Component: | khtml ecma | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | florian.reinhard, zahl |
Priority: | NOR | Keywords: | triaged |
Version: | 4.2.3 | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Janet
2009-05-21 10:48:14 UTC
https://wb1.ubs.com/fin/pub/gvu/quotes/markets_instruments/ Might be a better choice of url, at least that's where I found it. I see the same error given in the report above, as in the js "bug" dialog. 4.2.86 & tested with UA for ie 7.0 heh... pretending to be chrome on XP didn't help either. Can be fixed by: --- xml/dom_textimpl.cpp (revision 967193) +++ xml/dom_textimpl.cpp (working copy) @@ -304,7 +304,7 @@ DOMString CommentImpl::toString() const { // FIXME: substitute entity references as needed! - return DOMString("<!--") + escapeHTML( nodeValue() ) + "-->"; + return DOMString("<!--") + nodeValue() + "-->"; } // --------------------------------------------------------------------------- ... Now I just need to bundle this with an another innerHTML change I have pending. Any news on this? The page still isn't viewable with konqueror of KDE 4.3.4. SVN commit 1070771 by orlovich: We don't want to escape comments' innerHTML --- websites do silly string-munching strings with it; and HTML5 prescribes this behavior, too. BUG: 193463 M +12 -12 dom_textimpl.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1070771 SVN commit 1070774 by orlovich: Regression test for #193463 CCBUG:193463 A baseline/dom/comment-innerHTML.html-dom AM baseline/dom/comment-innerHTML.html-dump.png A baseline/dom/comment-innerHTML.html-render A tests/dom/comment-innerHTML.html WebSVN link: http://websvn.kde.org/?view=rev&revision=1070774 |