<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://svn.reviewboard.kde.org/r/5823/">http://svn.reviewboard.kde.org/r/5823/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On November 10th, 2010, 5:30 p.m., <b>Chani Armitage</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">no, I don't think this is correct.
check out line 880:
ImmutabilityType Applet::immutability() const
that function doesn't return d->immutability, but the strongest of that and various other checks. and it suggests that Corona::immutability() should be responsible for checking kiosk things.
I'm not sure what checkImmutability is supposed to be doing, but setting d->immutability isn't it ;)
...also note that, once you set SystemImmutable, you can't unset it. at least that's how the code read to me. so test that kiosk can both lock *and* unlock things ;)</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">i think both are ssafe, i.e if the immutability isn't set, the proper (aka more restrictive) one should always be returned.
setting it also at applet level is like to be uhm more sure about it.
systemimmutable wasn't supposed to be changed at runtime in any way, and after the patch still isn't
thinking about it, a problem of the patch could be that now the systemimmutable ness would be written into the config file, so wouldn't be possible to unlock everything by just changing from kdeglobals (with kde not running) but the change would have to be propagated by hand into plasma-desktop-appletsrc</pre>
<br />
<p>- Marco</p>
<br />
<p>On November 10th, 2010, 4:39 p.m., Lubos Lunak wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/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 Plasma and Marco Martin.</div>
<div>By Lubos Lunak.</div>
<p style="color: grey;"><i>Updated 2010-11-10 16:39:31</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;">Probably as a result of r800724, Plasma doesn't seem to follow Kiosk restrictions at all (just grep the sources for where SystemImmutable is assigned to something - it isn't). This patch may possibly be a suitable fix.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>trunk/KDE/kdelibs/plasma/applet.cpp <span style="color: grey">(1182745)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/5823/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>