<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/106618/">http://git.reviewboard.kde.org/r/106618/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdelibs.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated Sept. 28, 2012, 4:54 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This is a user contributed patch that changes the "Treat all cookies as session cookies" global option into a per domain/cookie option so that the policy can be configured in a more fine grained fashion. It was initially intended to be push upstream for the 4.6 release, but for reasons I do not want to restate again, see http://lists.kde.org/?t=132070395200003&r=1&w=2 if you want the details, the patch never made it in.
Well I want to push this into kdelibs for the 4.10 release before it is too late. The patch comes complete with unit tests that exercise the new changes and was contributed by Gregorio Guidi. He did all the work and made all the necessary fixes based on feedback from me. I only added code to update the kcookiejar config file and modified a couple of unrelated minor GUI issues. As such I want to reconginze this effort by pushing this patch on his behalf.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=54300">54300</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kioslave/http/kcookiejar/CMakeLists.txt <span style="color: grey">(e5ff3ee)</span></li>
<li>kioslave/http/kcookiejar/kcookiejar.h <span style="color: grey">(896cab7)</span></li>
<li>kioslave/http/kcookiejar/kcookiejar.cpp <span style="color: grey">(db4606c)</span></li>
<li>kioslave/http/kcookiejar/kcookiescfg.pl <span style="color: grey">(PRE-CREATION)</span></li>
<li>kioslave/http/kcookiejar/kcookiescfg.upd <span style="color: grey">(3c1cd02)</span></li>
<li>kioslave/http/kcookiejar/kcookieserver.cpp <span style="color: grey">(c8a61fb)</span></li>
<li>kioslave/http/kcookiejar/kcookiewin.h <span style="color: grey">(ce61c5a)</span></li>
<li>kioslave/http/kcookiejar/kcookiewin.cpp <span style="color: grey">(0713cdd)</span></li>
<li>kioslave/http/kcookiejar/tests/cookie_session.test <span style="color: grey">(PRE-CREATION)</span></li>
<li>kioslave/http/kcookiejar/tests/cookie_settings.test <span style="color: grey">(7fc1a03)</span></li>
<li>kioslave/http/kcookiejar/tests/kcookiejartest.cpp <span style="color: grey">(5dbe2ca)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106618/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/106618/s/736/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/09/28/new_cookiejar_ask_for_confirmation_dialog_400x100.png" style="border: 1px black solid;" alt="new cookie confirmation dialog" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>