<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 29/05/15 04:26, Daniel van Vugt
wrote:<br>
</div>
<blockquote cite="mid:5567DC60.1070602@canonical.com" type="cite">Sorry
for the confusion. Yes it's just a bug that needs fixing...
<br>
<br>
If one frame has been posted then you need to take it because you
can never predict how long before a second one arrives, if at all.
<br>
</blockquote>
<br>
For clarity there are multiple bugs:<br>
<br>
/1/ We composite when a surface is created - this leads to a nested
server posting a buffer with no content.<br>
/2/ USC throws a buffer away as a workaround. Throwing a buffer away
is also a bug.<br>
<br>
Having looked at the LegacySceneChangeNotification code it is simple
to fix the specific case of composite when a surface is created.
However, <br>
<br>
/3/ there are a host of cases in LegacySurfaceChangeNotification
where we would composite unnecessarily for surfaces that are not
visible. For example, the mir_demo_client_basic example in the
original post triggers composition when a surface that has never
posted and is not visible is destroyed.<br>
<br>
I'll MP some improvements to Legacy*ChangeNotification to address
/1/ and /3/ and then remove the workaround from USC.<br>
<br>
The only disagreement on this thread seems to be whether or not
mir_demo_server should clear the background when started. I propose
to take the easier approach of retaining the existing behaviour (of
not).<br>
<meta http-equiv="content-type" content="text/html;
charset=windows-1252">
</body>
</html>