<html> <head> <base href="https://bugzilla.gnome.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - wayland: possible infinite loop in gdkwindow-wayland" href="https://bugzilla.gnome.org/show_bug.cgi?id=759299#c8">Comment # 8</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - wayland: possible infinite loop in gdkwindow-wayland" href="https://bugzilla.gnome.org/show_bug.cgi?id=759299">bug 759299</a> from <a href="page.cgi?id=describeuser.html&login=ofourdan%40redhat.com" title="Olivier Fourdan <ofourdan@redhat.com>"> Olivier Fourdan</a> <pre>Created <a href="attachment.cgi?id=317426&action=diff" name="attach_317426" title="wayland: Check transient loop">attachment 317426</a> <a href="attachment.cgi?id=317426&action=edit" title="wayland: Check transient loop">[details]</a> <a href='review?bug=759299&attachment=317426'>[review]</a> wayland: Check transient loop Note, this patch tries to avoid the problem by setting the transient relationship before checking for a possible loop. If such a loop is found, the previous transient value is restored. Yet, because unmap_subsurface() is called first and also uses transient_for, we need to proceed till the end of the function gdk_wayland_window_set_transient_for() as if we modified the transient, even if the value is not changed so that the subsurface has a chance to be recreated if needed.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> </ul> </body> </html>