<div dir="ltr"><div>I hope to look into it soon.<br><br></div>-Sonal<br><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 14, 2015 at 10:52 AM, Olivier Fourdan <span dir="ltr"><<a href="mailto:fourdan@gmail.com" target="_blank">fourdan@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sonal,<br>
<br>
The implementation I pointed out is actually much simpler than yours,<br>
and you get GdkPixbuf as well, I reckon it's much better and efficient<br>
than using XGetImage().<br>
<br>
Cheers<br>
<span><font color="#888888">Olivier<br>
</font></span><div><div><br>
<br>
On 13 January 2015 at 15:52, Olivier Fourdan <<a href="mailto:fourdan@gmail.com" target="_blank">fourdan@gmail.com</a>> wrote:<br>
> Nope, sorry.<br>
><br>
> Olivier<br>
><br>
> On 13 January 2015 at 15:08, Sonal Santan <<a href="mailto:sonal.santan@gmail.com" target="_blank">sonal.santan@gmail.com</a>> wrote:<br>
>> Hello,<br>
>><br>
>> IMHO this implementation fits very well with the existing infrastructure in<br>
>> tabwin which expects a GdkPixbuf to draw onto the window buttons, requiring<br>
>> minimal changes overall. I do agree that in an ideal world we would not use<br>
>> XGetImage to capture the contents of the preview pixmap but directly render<br>
>> the preview over a picture provided by the preview client. But that would<br>
>> require more changes. Also, I believe this implementation is inline with my<br>
>> original proposal of using XImage as a mechanism to exchange previews to<br>
>> clients. The images are small so the overhead should not be huge.<br>
>><br>
>> Can we go ahead with the current design for now while I will look at<br>
>> optimizing the implementation as a future step?<br>
>><br>
>> -Sonal<br>
>><br>
>> On Mon, Jan 12, 2015 at 10:10 AM, Olivier Fourdan <<a href="mailto:fourdan@gmail.com" target="_blank">fourdan@gmail.com</a>> wrote:<br>
>>><br>
>>> Hi<br>
>>><br>
>>> I haven't looked much at the code, but using XGetImage and XImage<br>
>>> conversion seem inappropriate here.<br>
>>><br>
>>> Using a compositor, all client windows are unredirected, meaning they<br>
>>> are rendered to an offscreen pixmap already, and the compositor has<br>
>>> all that's required already, it "knows" about all this so all you need<br>
>>> to do is to add hooks in the compositor to retrieve this and use that<br>
>>> picture, that's all. Metacity's compositor was based originally on<br>
>>> xfwm4's and had that feature up until not too long ago (not sure it's<br>
>>> still there), you should look there for an example.<br>
>>><br>
>>> Also, please note that the gtk_ prefix is for gtk alone.<br>
>>><br>
>>> Cheers,<br>
>>> Olivier<br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> Xfce4-dev mailing list<br>
>> <a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br>
>> <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
_______________________________________________<br>
Xfce4-dev mailing list<br>
<a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br>
<a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
</div></div></blockquote></div><br></div></div></div></div>