Bug 164204 - [GTK] Plugin process crash in WebKit::NetscapePluginX11::visibilityDidChange with evince browser plugin
Summary: [GTK] Plugin process crash in WebKit::NetscapePluginX11::visibilityDidChange ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-30 08:18 PDT by Michael Catanzaro
Modified: 2016-12-06 08:00 PST (History)
4 users (show)

See Also:


Attachments
Speculative fix (2.17 KB, patch)
2016-10-31 09:44 PDT, Carlos Garcia Campos
mcatanzaro: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2016-10-30 08:18:19 PDT
An Evince browser plugin crash. Full backtrace downstream as usual. Truncated backtrace:

Thread no. 1 (10 frames)
 #0 gdk_window_has_impl at gdkwindow.c:626
 #1 _gdk_window_has_impl at gdkwindow.c:627
 #2 gdk_x11_window_get_xid at gdkwindow-x11.c:5542
 #3 WebKit::NetscapePluginX11::visibilityDidChange at /usr/src/debug/webkitgtk-2.12.3/Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:267
 #4 WebKit::PluginControllerProxy::visibilityDidChange at /usr/src/debug/webkitgtk-2.12.3/Source/WebKit2/PluginProcess/PluginControllerProxy.cpp:437
 #5 IPC::callMemberFunctionImpl<WebKit::PluginControllerProxy, void (WebKit::PluginControllerProxy::*)(bool), std::tuple<bool>, 0ul>(WebKit::PluginControllerProxy*, void (WebKit::PluginControllerProxy::*)(bool), std::tuple<bool>&&, std::index_sequence<0ul>) at /usr/src/debug/webkitgtk-2.12.3/Source/WebKit2/Platform/IPC/HandleMessage.h:16
 #6 IPC::callMemberFunction<WebKit::PluginControllerProxy, void (WebKit::PluginControllerProxy::*)(bool), std::tuple<bool>, std::make_index_sequence<1ul> >(std::tuple<bool>&&, WebKit::PluginControllerProxy*, void (WebKit::PluginControllerProxy::*)(bool)) at /usr/src/debug/webkitgtk-2.12.3/Source/WebKit2/Platform/IPC/HandleMessage.h:22
 #7 IPC::handleMessage<Messages::PluginControllerProxy::MutedStateChanged, WebKit::PluginControllerProxy, void (WebKit::PluginControllerProxy::*)(bool)> at /usr/src/debug/webkitgtk-2.12.3/Source/WebKit2/Platform/IPC/HandleMessage.h:92
 #8 WebKit::PluginControllerProxy::didReceivePluginControllerProxyMessage at /usr/src/debug/webkitgtk-2.12.3/x86_64-redhat-linux-gnu/DerivedSources/WebKit2/PluginControllerProxyMessageReceiver.cpp:53
 #9 WebKit::WebProcessConnection::didReceiveMessage at /usr/src/debug/webkitgtk-2.12.3/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:141
Comment 1 Michael Catanzaro 2016-10-30 08:21:24 PDT
Here's a second one: https://bugzilla.gnome.org/show_bug.cgi?id=762838

That one is surely an evince problem though.
Comment 2 Carlos Garcia Campos 2016-10-31 06:59:49 PDT
For some reason the plug is not embedded when NetscapePluginX11::visibilityDidChange is called. Do you remember if you were doing any tab drag an drop when this happened? Or was it after closing the tab?
Comment 3 Michael Catanzaro 2016-10-31 09:15:07 PDT
I don't remember, unfortunately I originally reported this in June and it just didn't make it upstream.
Comment 4 Carlos Garcia Campos 2016-10-31 09:44:30 PDT
Created attachment 293430 [details]
Speculative fix
Comment 5 WebKit Commit Bot 2016-10-31 09:45:55 PDT
This patch modifies one of the wasm.json files. Please ensure that any changes in one have been mirrored to the other. You can find the wasm.json files at "Source/JavaScriptCore/wasm/wasm.json" and "JSTests/wasm/wasm.json".
Comment 6 Carlos Garcia Campos 2016-11-02 02:44:46 PDT
Committed r208272: <http://trac.webkit.org/changeset/208272>