From 7827d7135628a2c9f20a4af33f396cd255dce15e Mon Sep 17 00:00:00 2001 From: Allan Sandfeld Jensen Date: Tue, 28 Nov 2017 15:14:38 +0100 Subject: Fix crash on exit-fullscreen using context menu MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The context menu does not like when we delete current widget while handling the context menu actions. Change-Id: I18dc9d19ae2a669c97c9d4bc833950fcdc0204b0 Reviewed-by: Michael Brüning --- src/webenginewidgets/api/qwebenginepage.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/webenginewidgets') diff --git a/src/webenginewidgets/api/qwebenginepage.cpp b/src/webenginewidgets/api/qwebenginepage.cpp index 9d8d52886..75ec44b5d 100644 --- a/src/webenginewidgets/api/qwebenginepage.cpp +++ b/src/webenginewidgets/api/qwebenginepage.cpp @@ -1325,7 +1325,8 @@ void QWebEnginePage::triggerAction(WebAction action, bool) d->adapter->inspectElementAt(menuData.position()); break; case ExitFullScreen: - d->adapter->exitFullScreen(); + // See under ViewSource, anything that can trigger a delete of the current view is dangerous to call directly here. + QTimer::singleShot(0, this, [d](){ d->adapter->exitFullScreen(); }); break; case RequestClose: d->adapter->requestClose(); -- cgit v1.2.3