Review Request: Add KDE CookieJar integration class to accessmanager.[h|cpp]

Urs Wolfer uwolfer at kde.org
Sat Oct 24 14:51:31 BST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1711/#review2794
-----------------------------------------------------------

Ship it!


IMHO you can apply this patch in trunk.

- Urs


On 2009-10-18 21:23:05, adawit wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1711/
> -----------------------------------------------------------
> 
> (Updated 2009-10-18 21:23:05)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> This patch does the following:
> 
> * It adds a nested namespace, NetworkIntegtation, to delinate KIO classes intended to be used for KDE integration only.
> * It adds a new class, KIO::NetworkIntegration::CookieJar, that allows Qt modules like QtWebKit to use KDE's cookiejar.
> * It documents everything including a note about the intended usage of these classes.
> 
> 
> Diffs
> -----
> 
>   /branches/KDE/4.3/kdelibs/kio/kio/accessmanager.h 1036185 
>   /branches/KDE/4.3/kdelibs/kio/kio/accessmanager.cpp 1036185 
> 
> Diff: http://reviewboard.kde.org/r/1711/diff
> 
> 
> Testing
> -------
> 
> This code was already proven to work in the kdewebkit module.
> 
> 
> Thanks,
> 
> adawit
> 
>





More information about the kde-core-devel mailing list