<div dir="ltr">Hi.<br><div><br>One more kind ping on this. It has been sent quite a long time ago without any reply. It would really appreciate if someone could let me know if this is a correct mailing list or not?</div><div>Thanks!<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-09-21 10:06 GMT-04:00 Nikolay Martynov <span dir="ltr"><<a href="mailto:mar.kolya@gmail.com" target="_blank">mar.kolya@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi.<br>
<br>
Just a kind ping. Is any additional information needed in this patch?<br>
<br>
Thanks!<br>
<div class="HOEnZb"><div class="h5"><br>
2017-09-13 23:23 GMT-04:00 Nikolay Martynov <<a href="mailto:mar.kolya@gmail.com">mar.kolya@gmail.com</a>>:<br>
> It looks like offsets calculated during image censoring are wrong.<br>
> This results in black (empty) images returns.<br>
><br>
> This fix is very similar to 6c6f09aac7f1d1367a042087b7681c<wbr>7fdf1d1e0f<br>
> that was applied to XGetImage<br>
><br>
> Visually this fixes chromium/firefox window sharing in multiscreen<br>
> configurations - without this patch most of the windows on 'secodnary'<br>
> screens are black.<br>
><br>
> This also should fix <a href="https://bugs.freedesktop.org/show_bug.cgi?id=101730" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=101730</a>.<br>
><br>
> Signed-off-by: Nikolay Martynov <<a href="mailto:mar.kolya@gmail.com">mar.kolya@gmail.com</a>><br>
> ---<br>
>  Xext/shm.c | 8 ++------<br>
>  1 file changed, 2 insertions(+), 6 deletions(-)<br>
><br>
> diff --git a/Xext/shm.c b/Xext/shm.c<br>
> index 91ea90b14..56555c7aa 100644<br>
> --- a/Xext/shm.c<br>
> +++ b/Xext/shm.c<br>
> @@ -650,9 +650,8 @@ ProcShmGetImage(ClientPtr client)<br>
>                 wBorderWidth((WindowPtr) pDraw) + (int) pDraw->height)<br>
>              return BadMatch;<br>
>          visual = wVisual(((WindowPtr) pDraw));<br>
> -        pVisibleRegion = NotClippedByChildren((<wbr>WindowPtr) pDraw);<br>
> -        if (pVisibleRegion)<br>
> -            RegionTranslate(<wbr>pVisibleRegion, -pDraw->x, -pDraw->y);<br>
> +        if (pDraw->type == DRAWABLE_WINDOW)<br>
> +            pVisibleRegion = &((WindowPtr) pDraw)->borderClip;<br>
>      }<br>
>      else {<br>
>          if (stuff->x < 0 ||<br>
> @@ -715,9 +714,6 @@ ProcShmGetImage(ClientPtr client)<br>
>          }<br>
>      }<br>
><br>
> -    if (pVisibleRegion)<br>
> -        RegionDestroy(pVisibleRegion);<br>
> -<br>
>      if (client->swapped) {<br>
>          swaps(&xgi.sequenceNumber);<br>
>          swapl(&xgi.length);<br>
> --<br>
> 2.11.0<br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Martynov Nikolay.<br>
Email: <a href="mailto:mar.kolya@gmail.com">mar.kolya@gmail.com</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Martynov Nikolay.<br>Email: <a href="mailto:mar.kolya@gmail.com" target="_blank">mar.kolya@gmail.com</a><br></div></div>
</div>