1097592 - [gtk3] freeze in gdk_x11_window_set_user_time()

Status: RESOLVED FIXED

(Core :: Widget: Gtk, defect)

Description

[Martin Stránský [:stransky] (ni? me)]

After some time running Firefox freezes in startup notification code, resp. in gdk_x11_window_set_user_time() where gdk_drawable_get_display() fails to get the display.

3716	void
3717	gdk_x11_window_set_user_time (GdkWindow *window,
3718	                              guint32    timestamp)
3719	{
3720	  GdkDisplay *display;
3721	  GdkDisplayX11 *display_x11;
3722	  GdkToplevelX11 *toplevel;
3723	  glong timestamp_long = (glong)timestamp;
3724	  Window xid;
3725	
3726	  if (GDK_WINDOW_DESTROYED (window) ||
3727	      !WINDOW_IS_TOPLEVEL_OR_FOREIGN (window))
3728	    return;
3729	
3730	  display = gdk_drawable_get_display (window);
3731	  display_x11 = GDK_DISPLAY_X11 (display); <<< display is null here

Comment 1

[Martin Stránský [:stransky] (ni? me)]

Attachment: _back

backtrace

Comment 2

[Martin Stránský [:stransky] (ni? me)]

The problem is PR_FindFunctionSymbolAndLibrary() in SetUserTimeAndStartupIDForActivatedWindow(). 

Sometimes a gtk2 library is choosen (like libflashplugin.so) so gdk_x11_window_set_user_time() from it freezes with null display.

Comment 3

[Martin Stránský [:stransky] (ni? me)]

Attachment: patch

Comment 4

[Karl Tomlinson (:karlt)]

Comment on attachment 8522971 [details] [diff] [review]
patch

The change is good, thanks, but I don't know that it will fix all related problems.
PR_FindFunctionSymbolAndLibrary() only considers libraries already loaded.
If GTK2 and GTK3 libraries are both loaded in the process, then there may be other problems.  Or perhaps this works because ld.so always prefers symbols from the first library loaded.

Is this from loading libflashplugin.so in the main process to check its MIME type?  Is it not unloaded after that happens?
Bug 939796 may be relevant.

Comment 5

[Martin Stránský [:stransky] (ni? me)]

IMHO it comes from the Bug 939796 because mime types are still loaded in-process. But I haven't see any other conflicts yet.

Comment 6

[Carsten Book [:Tomcat]]

Hi Martin,

could you post a link to  try run, thanks!

Comment 7

[Martin Stránský [:stransky] (ni? me)]

The try is here: https://tbpl.mozilla.org/?tree=Try&rev=60e107e0dea0

Comment 8

[Martin Stránský [:stransky] (ni? me)]

Looks okay (at least the modified gtk parts).

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/de71b9d01773

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/de71b9d01773