[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdelibs/khtml/ecma
From:       Maks Orlovich <maksim () kde ! org>
Date:       2008-03-03 22:54:46
Message-ID: 1204584886.377709.7191.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 781941 by orlovich:

automatically merged revision 781940:
Properly normalize whitespace in URLs when getting from JS.
Fixes article links @ http://milliyet.com.tr/
(Thanks to cartman for the report)

 M  +2 -2      kjs_html.cpp  


--- trunk/KDE/kdelibs/khtml/ecma/kjs_html.cpp #781940:781941
@@ -35,8 +35,8 @@
 #include <html/html_canvasimpl.h>
 #include <dom/dom_exception.h>
 
+#include <css/csshelper.h> // for parseUrl
 
-// ### HACK
 #include <html/html_baseimpl.h>
 #include <html/html_documentimpl.h>
 #include <html/html_formimpl.h>
@@ -1577,7 +1577,7 @@
 
 QString KJS::HTMLElement::getURLArg(unsigned id) const
 {
-  DOMString rel = impl()->getAttribute(id);
+  DOMString rel = khtml::parseURL(impl()->getAttribute(id));
   return !rel.isNull() ? impl()->getDocument()->completeURL(rel.string()) : QString();
 }
 
[prev in list] [next in list] [prev in thread] [next in thread]