[PATCH weston v2] ivi-shell: Damage view below after unmapping

[zou lan]

Hi  Emre

I have a question about this change:

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.

Best Regards
Nancy

2017-02-07 21:04 GMT+08:00 Pekka Paalanen <ppaalanen at gmail.com>:

> On Tue, 7 Feb 2017 12:55:59 +0000
> "Ucan, Emre (ADITG/SW1)" <eucan at de.adit-jv.com> wrote:
>
> > If ivilayer or ivisurf of ivi_view is made invisible in the
> > commit_changes call, we have to damage the weston_view below this
> > ivi_view. Otherwise content of this ivi_view will stay visible.
> >
> > Signed-off-by: Emre Ucan <eucan at de.adit-jv.com>
> > ---
> >  ivi-shell/ivi-layout.c |   13 ++++++++++++-
> >  1 file changed, 12 insertions(+), 1 deletion(-)
> >
> > diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c
> > index 712cc30..64e4ead 100644
> > --- a/ivi-shell/ivi-layout.c
> > +++ b/ivi-shell/ivi-layout.c
> > @@ -681,8 +681,19 @@ commit_changes(struct ivi_layout *layout)
> >                * If the view's layer or surface is invisible, we do not
> need
> >                * to update its properties.
> >                */
> > -             if (!ivilayer->prop.visibility ||
> !ivisurf->prop.visibility)
> > +             if (!ivilayer->prop.visibility ||
> !ivisurf->prop.visibility) {
> > +                     /*
> > +                     * If ivilayer or ivisurf of ivi_view is made
> invisible
> > +                     * in this commit_changes call, we have to damage
> > +                     * the weston_view below this ivi_view. Otherwise
> content
> > +                     * of this ivi_view will stay visible.
> > +                     */
> > +                     if ((ivilayer->prop.event_mask |
> ivisurf->prop.event_mask) &&
> > +                         IVI_NOTIFICATION_VISIBILITY)
> > +                             weston_view_damage_below(ivi_view->view);
> > +
> >                       continue;
> > +             }
> >
> >               update_prop(ivi_view);
> >       }
>
> Hi,
>
> looks fine to me, pushed:
>    19222b4..7fe0bb2  master -> master
>
>
> Thanks,
> pq
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20180410/cc15b868/attachment.html>