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

List:       kde-core-devel
Subject:    Re: kdewebkit moved to kdereview
From:       Andrea Diamantini <adjam7 () gmail ! com>
Date:       2009-10-28 14:44:16
Message-ID: 200910281544.16478.adjam7 () gmail ! com
[Download RAW message or body]

On Monday 26 October 2009 20:38:08 Dawit A. wrote:
> On Monday 26 October 2009 02:24:29 Benjamin Meyer wrote:
> > [snipped]
> > 
> > There seems to be missing convenience functions to get the kde network
> > access manager and plugin factory
> 
> Ahem... I do not follow... There is already a
>  QWebPage::networkAccessManager() and  QWebPage::pluginFactory().
> 
> KWebPluginFactory is a pure re-implementation of existing virtual member
> functions and does not provide any additional methods. For the network
>  access manager, if you invoked QWebPage::setNetworkAccessManager(...) with
>  an instance to a re-implementation, then you already have access to that
>  instance or you should simply use qobject_cast to down cast what is
>  returned by QWebPage::networkAccessManager....  So I fail to understand
>  what you mean here...

That's surely true, but as the KWebPage class is not intended to be used with 
usual Qt AccessManager and PluginFactory, but with our implementations, I 
think it's normal having the getter functions reimplemented.

I'd also like pointing out that a lot of people here is referring to this 
library saying just "kwebkit".
I think this is a better name for it (and its eventual namespace) than 
"kdewebkit". Just more KDE-ish. (change c-k: konqueror, konquest; prepend a k: 
khtml, kmail)


-- 
Andrea Diamantini, adjam
GPG Fingerprint: 57DE 8E32 7D1A 0E16 AA52 59D8 84F9 3ECD DBF9 730F

rekonq project
WEB: http://rekonq.sourceforge.net
IRC: rekonq@freenode

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