[prev in list] [next in list] [prev in thread] [next in thread]
List: kwin
Subject: Re: Review Request: Add Rule/Property based composite blocking
From: Thomas_Lübking <thomas.luebking () web ! de>
Date: 2011-03-20 17:53:16
Message-ID: 20110320175316.14129.88274 () vidsolbach ! de
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
> On March 20, 2011, 4:31 p.m., Martin Gräßlin wrote:
> > Very nice functionality, I like it.
> >
> > Probably we should extend it to
> > * announce support in _NET_SUPPORTED
> > * add functionality to netwm/KWindowSystem
> > * bring it to NETWM spec (though I doubt we have success given that GNOME Shell \
> > and Unity require compositing all the time)
Afaik unity will explicitly NOT require compositing (at all) - therefore the move \
towards Qt. However i'm not too optimistic that it will be accepted (outside KDE) \
either. We might have to think about a (partial) logic inversion.
In that case, every fullscreen client would block compositing unless explicitly \
permitting it.
So _KDE_NET_WM_BLOCK_COMPOSITING would turn into _KDE_NET_WM_WANT_COMPOSITING
0 means "no", everything else "yes" and an absent property triggers the default \
behavior (not fs -> compositing good, fs -> compositing bad)
This way we'd catch "legal" fs clients (games, video players) out of the box, nasty \
(tm ;-) gkt+ applications (office/browser) look "ugly" in fs while fs prone KDE \
applications (<strike>koffice</strike> calligra, konqueror/rekonq i assume? - \
whatever) could keep compositing active despite the fs mode.
I'd rather dislike this way, but ultimately it's the result on the user desktop that \
matters... :-\
=> I'll branch off the netwm stuff, we try and on failure can still think about the \
more aggressive push.
> On March 20, 2011, 4:31 p.m., Martin Gräßlin wrote:
> > kwin/client.cpp, lines 2114-2117
> > <http://git.reviewboard.kde.org/r/100893/diff/1/?file=11605#file11605line2114>
> >
> > Why the bool readProperty? If I read the code correctly it's a no-op if \
> > readProperty is false
Code body. If we don't (find a way to) do it state dependent (fs mode) it's \
completely superfluous. Otherwise just lacks "else blocks_compositing = \
rules()->checkBlockCompositing(blocks_compositing);"
- Thomas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100893/#review2063
-----------------------------------------------------------
On March 20, 2011, 3:11 p.m., Thomas Lübking wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100893/
> -----------------------------------------------------------
>
> (Updated March 20, 2011, 3:11 p.m.)
>
>
> Review request for kwin.
>
>
> Summary
> -------
>
> Allows windows (or users via the rule system) to block compositing while they're \
> present. The user still can resume and re-suspend compositing at any time (but if \
> it has originally been suspended for a blocking client it will be resumed the \
> moment the last blocking client disappears)
> Outstanding issue is that the rules to not allow to apply by fullscreen mode.
> No real idea how we should handle this (add to rules globally or do some rule \
> specific)
>
> Diffs
> -----
>
> kwin/atoms.h 7d5752c
> kwin/atoms.cpp 5e93b2a
> kwin/client.h 0d479a0
> kwin/client.cpp ec60bde
> kwin/composite.cpp 6783d32
> kwin/events.cpp 05cf864
> kwin/kcmkwin/kwinrules/ruleswidget.h ca44bf2
> kwin/kcmkwin/kwinrules/ruleswidget.cpp f9fc407
> kwin/kcmkwin/kwinrules/ruleswidgetbase.ui 74ac56b
> kwin/manage.cpp 135a016
> kwin/rules.h 87b2055
> kwin/rules.cpp 9e4136a
> kwin/workspace.h 3f6ad49
> kwin/workspace.cpp 3c77511
>
> Diff: http://git.reviewboard.kde.org/r/100893/diff
>
>
> Testing
> -------
>
> Rule based & "xprop -f _KDE_NET_WM_BLOCK_COMPOSITING 32a -set \
> _KDE_NET_WM_BLOCK_COMPOSITING 1" No client internal property setting tested (so \
> far) but it should work as well... ;-)
>
> Thanks,
>
> Thomas
>
>
[Attachment #5 (text/html)]
<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/100893/">http://git.reviewboard.kde.org/r/100893/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <p style="margin-top: 0;">On March 20th, 2011, 4:31 p.m., <b>Martin \
Gräßlin</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;">Very nice functionality, I like it.
Probably we should extend it to
* announce support in _NET_SUPPORTED
* add functionality to netwm/KWindowSystem
* bring it to NETWM spec (though I doubt we have success given that GNOME Shell and \
Unity require compositing all the time)</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;">Afaik unity will \
explicitly NOT require compositing (at all) - therefore the move towards Qt. However \
i'm not too optimistic that it will be accepted (outside KDE) either. We might \
have to think about a (partial) logic inversion.
In that case, every fullscreen client would block compositing unless explicitly \
permitting it.
So _KDE_NET_WM_BLOCK_COMPOSITING would turn into _KDE_NET_WM_WANT_COMPOSITING
0 means "no", everything else "yes" and an absent property \
triggers the default behavior (not fs -> compositing good, fs -> compositing \
bad)
This way we'd catch "legal" fs clients (games, video players) out of \
the box, nasty (tm ;-) gkt+ applications (office/browser) look "ugly" in fs \
while fs prone KDE applications (<strike>koffice</strike> calligra, \
konqueror/rekonq i assume? - whatever) could keep compositing active despite the fs \
mode.
I'd rather dislike this way, but ultimately it's the result on the user \
desktop that matters... :-\
=> I'll branch off the netwm stuff, we try and on failure can still think \
about the more aggressive push.</pre> <br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <p style="margin-top: 0;">On March 20th, 2011, 4:31 p.m., <b>Martin \
Gräßlin</b> wrote:</p> <blockquote style="margin-left: 1em; border-left: 2px solid \
#d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/100893/diff/1/?file=11605#file11605line2114" \
style="color: black; font-weight: bold; text-decoration: \
underline;">kwin/client.cpp</a> <span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">void \
Client::updateCursor()</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">2114</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="kt">void</span> <span class="n">Client</span><span class="o">::</span><span \
class="n">updateCompositeBlocking</span><span class="p">(</span><span \
class="kt">bool</span> <span class="n">readProperty</span><span \
class="p">)</span></pre></td> </tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">2115</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="p">{</span></pre></td> </tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">2116</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="k">const</span> <span class="kt">bool</span> <span \
class="n">usedToBlock</span> <span class="o">=</span> <span \
class="n">blocks_compositing</span><span class="p">;</span></pre></td> </tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">2117</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="k">if</span> <span class="p">(</span><span class="n">readProperty</span><span \
class="p">)</span> <span class="p">{</span></pre></td> </tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Why the bool \
readProperty? If I read the code correctly it's a no-op if readProperty is \
false</pre> </blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Code body. \
If we don't (find a way to) do it state dependent (fs mode) it's completely \
superfluous. Otherwise just lacks "else blocks_compositing = \
rules()->checkBlockCompositing(blocks_compositing);"</pre> <br />
<p>- Thomas</p>
<br />
<p>On March 20th, 2011, 3:11 p.m., Thomas Lübking wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('http://git.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 kwin.</div>
<div>By Thomas Lübking.</div>
<p style="color: grey;"><i>Updated March 20, 2011, 3:11 p.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;">Allows windows (or users via the rule system) to block compositing while \
they're present. The user still can resume and re-suspend compositing at any time \
(but if it has originally been suspended for a blocking client it will be resumed the \
moment the last blocking client disappears)
Outstanding issue is that the rules to not allow to apply by fullscreen mode.
No real idea how we should handle this (add to rules globally or do some rule \
specific)</pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Rule based & "xprop -f _KDE_NET_WM_BLOCK_COMPOSITING 32a -set \
_KDE_NET_WM_BLOCK_COMPOSITING 1" No client internal property setting tested (so \
far) but it should work as well... ;-)</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>kwin/atoms.h <span style="color: grey">(7d5752c)</span></li>
<li>kwin/atoms.cpp <span style="color: grey">(5e93b2a)</span></li>
<li>kwin/client.h <span style="color: grey">(0d479a0)</span></li>
<li>kwin/client.cpp <span style="color: grey">(ec60bde)</span></li>
<li>kwin/composite.cpp <span style="color: grey">(6783d32)</span></li>
<li>kwin/events.cpp <span style="color: grey">(05cf864)</span></li>
<li>kwin/kcmkwin/kwinrules/ruleswidget.h <span style="color: \
grey">(ca44bf2)</span></li>
<li>kwin/kcmkwin/kwinrules/ruleswidget.cpp <span style="color: \
grey">(f9fc407)</span></li>
<li>kwin/kcmkwin/kwinrules/ruleswidgetbase.ui <span style="color: \
grey">(74ac56b)</span></li>
<li>kwin/manage.cpp <span style="color: grey">(135a016)</span></li>
<li>kwin/rules.h <span style="color: grey">(87b2055)</span></li>
<li>kwin/rules.cpp <span style="color: grey">(9e4136a)</span></li>
<li>kwin/workspace.h <span style="color: grey">(3f6ad49)</span></li>
<li>kwin/workspace.cpp <span style="color: grey">(3c77511)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100893/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
_______________________________________________
kwin mailing list
kwin@kde.org
https://mail.kde.org/mailman/listinfo/kwin
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic