<html>
    <head>
      <base href="https://bugzilla.gnome.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - [Wayland] Crash under gdk_wayland_window_attach_image()"
   href="https://bugzilla.gnome.org/show_bug.cgi?id=793062#c18">Comment # 18</a>
              on <a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - [Wayland] Crash under gdk_wayland_window_attach_image()"
   href="https://bugzilla.gnome.org/show_bug.cgi?id=793062">bug 793062</a>
              from <span class="vcard"><a href="page.cgi?id=describeuser.html&login=ofourdan%40redhat.com" title="Olivier Fourdan <ofourdan@redhat.com>"> <span class="fn">Olivier Fourdan</span></a>
</span></b>
        <pre>(In reply to Martin Stransky from <a href="show_bug.cgi?id=793062#c16">comment #16</a>)
<span class="quote">> With the latest patch applied I still see a crash:</span >

Weird. Can you double-check the version you use is the one with the patch?

Reason I ask is because the backtrace in <a href="show_bug.cgi?id=793062#c16">comment #16</a> shows:

 #15 _gdk_window_process_updates_recurse_helper
 at gdkwindow.c line 3853

with:

 state = (GDK_WINDOW_STATE_WITHDRAWN | GDK_WINDOW_STATE_FOCUSED)

However the patch calls the _gdk_event_emit() only if
GDK_WINDOW_IS_MAPPED(window).

GDK_WINDOW_IS_MAPPED() is defined as:

  #define GDK_WINDOW_IS_MAPPED(window) (((window)->state &
GDK_WINDOW_STATE_WITHDRAWN) == 0)

So with the patch applied you cannot get to frame #15 with “state =
(GDK_WINDOW_STATE_WITHDRAWN | GDK_WINDOW_STATE_FOCUSED)”

(In reply to Martin Stransky from <a href="show_bug.cgi?id=793062#c17">comment #17</a>)
<span class="quote">> I can 100% reproducible with the firefox-59 build I provide. Navigate to url
> bar and type "about:addons" there. When awesomebar disappears FF crashes.</span >

Not here, with the patch applied, it works.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>