<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Dec 9, 2016 at 8:40 PM, Michael Biebl <span dir="ltr"><<a target="_blank" href="mailto:biebl@debian.org">biebl@debian.org</a>></span> wrote:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">Hi Daniel<br>
<span class="gmail-"><br>
</span><span class="gmail-"><br>
</span>I see that you've been working with upstream, trying to resolve this.<br>
Thanks a lot for that.<br>
The latest stable release 3.22.5, contains your patches.<br>
<br>
What's unclear to me is whether your patches replace the above two<br>
patches and we can drop them now, or if we still need them (for<br>
Ubuntu/Unity).<br>
I've CCed William, from the Ubuntu side. Maybe you can confirm, that<br>
those patches are safe to drop now or if they are still needed.<br>
<br>
Regards,<br>
Michael<br>
<br></blockquote><div><br></div><div>Hi Michael,<br><br></div><div>It took a while, but we got there in the end. :) I'm glad to have done my bit to help, and to get William's fix into 3.22.<br><br></div><div>William's and my patches only resolve assertion failures, which were in part revealed by the Debian patches to prevent offscreen grabs:<br><br>William's patch fixed an oversight in GtkCellRendererCombo, where the trigger event was not passed and failed an upstream assertion. This was forgotten for cherry-picking to 3.22, so I did that as part of merging my patch from master. My patch was for a parallel issue: some dodgy ordering of menu grab vs realise in GtkComboBox (with particular sets of properties) - which caused an assertion in Debian patch 016 to fail.<br></div></div><br></div><div class="gmail_extra">So, if you want to prevent offscreen grabs, currently patches 016 and 017 seem to be still required. Near or identical patches are submitted upstream, and reviewed quite positively, but they don't seem to have gone any further than that. Maybe you'd like to request an update there?<br><br><a href="https://bugzilla.gnome.org/show_bug.cgi?id=658563">https://bugzilla.gnome.org/show_bug.cgi?id=658563</a><br><br></div><div class="gmail_extra">That's the thread with a reviewed patch against offscreen grabs for both widgets and devices, which AFAICT would cover both patches 016 and 017. It might be worth asking for an update, now that it wouldn't generate errors on Debian et al if merged.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Cheers!<br></div><div class="gmail_extra">Daniel<br><br></div></div>