Regression on page

[Maksim Orlovich]

> Dear all,
>
> since a few month now, I'm trying out KDE 4.1.1. on Kubuntu hardy.
> There are still many many bugs and it is much less usable than 3.5.9.
> My hope was to get all the advances in konqueror. But to my surprise,
> konqueror does NOT work as good in 4.1.1 and has really many
> regressions and crashes.

Thanks for the feedback.

>
> Take
> http://ipyorkshire.blogspot.com/2008/10/glossary-of-intellectual-property-terms.html
> in 3.5.9 it works like charm
> in 4.1.1 I get a blank canvas

I can't confirm the problem with ~4.1.2; the page shows up fine (though
the JS on the page suggests we should support [[Get]] of attribute name on
NamedNodeMap).

> Also the W3C slidemaker does not work in konqueror 4.1.1 but they
> don't work in 3.5.9 either. So I will ask back and see if the
> javascript stuff in there is responsible. Try
> http://www.w3.org/2008/Talks/0908-Matt-Geo-Widgets/

I can confirm that. The problem seem to be that we return .hash as
%-encoded, which confuses the slide number parser in the tool.
David, does the below look sane?

--- ecma/kjs_window.cpp (revision 865420)
+++ ecma/kjs_window.cpp (working copy)
@@ -2529,7 +2532,7 @@
   KUrl url = m_frame->m_part->url();
   switch(token) {
     case Hash:
-      return jsString( UString(url.ref().isNull() ? QString("") : '#' +
url.ref()) );
+      return jsString( UString(url.htmlRef().isNull() ? QString("") : '#'
+ url.htmlRef()) );
     case Host: {
       UString str = url.host();
       if (url.port() > 0)