<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 14 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> <p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Hi,<o:p></o:p></span></p> <p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p> <p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Are you asking if it is possible to move weston_view_damage_below() to commit_screen_list ? or are you asking why we are calling it ?<o:p></o:p></span></p> <p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p> <p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#1F497D">Best regards</span><span style="font-size:8.0pt;color:#1F497D"><br> </span><span style="font-size:8.0pt;font-family:"Arial","sans-serif";color:#1F497D"><br> </span><b><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#1F497D">Emre Ucan</span></b><span style="font-size:8.0pt;color:#1F497D"><br> </span><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#1F497D">Engineering Software Base (ADITG/ESB)</span><span style="font-size:8.0pt;color:#1F497D"><br> </span><span style="font-size:8.0pt;font-family:"Arial","sans-serif";color:#1F497D"><br> </span><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#1F497D">Tel. +49 5121 49 6937</span><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p></o:p></span></p> <div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt"> <div> <div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm"> <p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> zou lan [mailto:nancy.lan.zou@gmail.com] <br> <b>Sent:</b> Dienstag, 10. April 2018 08:55<br> <b>To:</b> Pekka Paalanen<br> <b>Cc:</b> Ucan, Emre (ADITG/ESB); wayland-devel@lists.freedesktop.org<br> <b>Subject:</b> Re: [PATCH weston v2] ivi-shell: Damage view below after unmapping<o:p></o:p></span></p> </div> </div> <p class="MsoNormal"><o:p> </o:p></p> <div> <p class="MsoNormal">Hi <span style="font-size:10.5pt;font-family:"Arial","sans-serif";color:#222222;background:white"> Emre</span><o:p></o:p></p> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal"><span style="font-size:10.5pt">I have a question about this change:</span><o:p></o:p></p> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal"><span style="font-size:10.5pt">Is the commit_screen_list function not enough to handle the layer/surface's visibility? Why need to handle visibility in commit_changes? They are called ivi_layout_commit_changes together.</span><o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal"><span style="font-size:10.5pt">Best Regards</span><o:p></o:p></p> </div> <div> <p class="MsoNormal"><span style="font-size:10.5pt">Nancy </span><o:p></o:p></p> </div> </div> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> <div> <p class="MsoNormal">2017-02-07 21:04 GMT+08:00 Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>>:<o:p></o:p></p> <div> <div> <p class="MsoNormal" style="margin-bottom:12.0pt">On Tue, 7 Feb 2017 12:55:59 +0000<br> "Ucan, Emre (ADITG/SW1)" <<a href="mailto:eucan@de.adit-jv.com">eucan@de.adit-jv.com</a>> wrote:<br> <br> > If ivilayer or ivisurf of ivi_view is made invisible in the<br> > commit_changes call, we have to damage the weston_view below this<br> > ivi_view. Otherwise content of this ivi_view will stay visible.<br> ><br> > Signed-off-by: Emre Ucan <<a href="mailto:eucan@de.adit-jv.com">eucan@de.adit-jv.com</a>><br> > ---<br> > ivi-shell/ivi-layout.c | 13 ++++++++++++-<br> > 1 file changed, 12 insertions(+), 1 deletion(-)<br> ><br> > diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c<br> > index 712cc30..64e4ead 100644<br> > --- a/ivi-shell/ivi-layout.c<br> > +++ b/ivi-shell/ivi-layout.c<br> > @@ -681,8 +681,19 @@ commit_changes(struct ivi_layout *layout)<br> > * If the view's layer or surface is invisible, we do not need<br> > * to update its properties.<br> > */<br> > - if (!ivilayer->prop.visibility || !ivisurf->prop.visibility)<br> > + if (!ivilayer->prop.visibility || !ivisurf->prop.visibility) {<br> > + /*<br> > + * If ivilayer or ivisurf of ivi_view is made invisible<br> > + * in this commit_changes call, we have to damage<br> > + * the weston_view below this ivi_view. Otherwise content<br> > + * of this ivi_view will stay visible.<br> > + */<br> > + if ((ivilayer->prop.event_mask | ivisurf->prop.event_mask) &&<br> > + IVI_NOTIFICATION_VISIBILITY)<br> > + weston_view_damage_below(ivi_view->view);<br> > +<br> > continue;<br> > + }<br> ><br> > update_prop(ivi_view);<br> > }<o:p></o:p></p> </div> </div> <p class="MsoNormal" style="margin-bottom:12.0pt">Hi,<br> <br> looks fine to me, pushed:<br> 19222b4..7fe0bb2 master -> master<br> <br> <br> Thanks,<br> pq<br> <br> _______________________________________________<br> wayland-devel mailing list<br> <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br> <a href="https://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">https://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><o:p></o:p></p> </div> <p class="MsoNormal"><o:p> </o:p></p> </div> </div> </div> </body> </html>