[patches to kdelibs/base] request for comment

[Marcel Partap]

Hi KDE devs,
first time I'm laying my hands on the KDE source, and proposing a change to kdelibs I thought I'd 
rather ask for opinions on this list, too, soo...
First patch fixes the 'problem' (#113077) that sometimes when you open a javascript popup in a tab, 
the back button takes you to a 'malformed url', by opening the final destination url in the first 
place. Why is it done differently now, seems redundant, no?
Second patch is more invasive as it adds a bool hasSubDomain() function to the KUrl API and makes 
upUrl go up on subdomains, too. I have had a quick look at all occurrencesof upUrl() in KDE and have 
not found any potential breakage.. is this ok for inclusion in 4.1? Next patch then only adds this 
visibly to konquerors up button (#95433)..
regards & happy saturday ;)
marcel.


-- 
<div id="signature">
   "Obstacles are those frightful things you see when you take your eyes off your goal."
                                                               -- Henry Ford (1863-1947)
   Change the world! Vote revolution: http://hfopi.org/vote-future
</div>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs-khtml-javascript-straightly-open-url-without-empty-history-entry.patch
Type: text/x-patch
Size: 1247 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080209/7ec9233a/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs-kurl-add-hassubdomain-and-enhance-upurl.patch
Type: text/x-patch
Size: 3058 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080209/7ec9233a/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdebase-konqueror-up-button-ascend-from-subdomain.patch
Type: text/x-patch
Size: 818 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080209/7ec9233a/attachment-0002.bin>