<div dir="ltr"><div>Hi Bill,</div><div> </div><div>"This is not controlled by a count, but by whether a window is already visible or already in the notification state. Clients should be able to send a lot of these in a row. They cannot reliably test if they are invisible and send the request only then, as there is a race condition."</div>
<div> </div><div>Thanks a lot for that ; your remark made me find a bug in my implementation, effectively when a surface was shown via a notification (i.e. tooltip), subsequent present() request would not re-create the notification. This is bad.</div>
<div> </div><div>Now fixed here : <a href="https://github.com/Tarnyko/weston-xdg_surface_present/commit/4265ab36f644d2663bf14d4b0d42fb8bcb6ca6f7">https://github.com/Tarnyko/weston-xdg_surface_present/commit/4265ab36f644d2663bf14d4b0d42fb8bcb6ca6f7</a></div>
<div> </div><div>Regards,</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-07-29 21:16 GMT+02:00 Bill Spitzak <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On 07/29/2014 11:40 AM, Manuel Bachmann wrote:<br>
<br>
<blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">
When creating a xdg_surface, the surface will not be mapped (i.e. shown)<br>
by desktop-shell anymore. It will only be if xdg_surface_present() has<br>
been called once.<br>
</blockquote>
<br></div>
There seems to be a design goal in Wayland to prevent clients from making surfaces that they never map. So it would be better if creation + commit of a surface did the same thing as present. Also this does not break existing clients.<br>

<br>
There is nothing special about the first time the surface wants attention (other than historical legacy). The desktop should be allowed to turn this into a notification just like it would on subsequent calls.<div><br>
<br>
<blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">
If called twice, or more, the request will send an event to<br>
desktop-shell, so it can display a notification.<br>
</blockquote>
<br></div>
This is not controlled by a count, but by whether a window is already visible or already in the notification state. Clients should be able to send a lot of these in a row. They cannot reliably test if they are invisible and send the request only then, as there is a race condition.<br>

<br>
I also think the term "present" is not a great idea. This should be exactly the same as "raise" or "show" or "activate" or any number of other terms, but I have never seen the word "present" used before. I would reuse an existing term. One reason is to prevent somebody else from adding a redundant api for that term, because they did not realize "present" is the thing they are looking for.<br>

______________________________<u></u>_________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.<u></u>freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/<u></u>mailman/listinfo/wayland-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr"><font>Regards,<br>
<br>
<i><b>Manuel BACHMANN</b><br>
Tizen Project<br>
VANNES-FR</i><br>
</font></div>
</div>