diff options
author | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2017-11-28 15:14:38 +0100 |
---|---|---|
committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2017-11-29 11:42:02 +0000 |
commit | 7827d7135628a2c9f20a4af33f396cd255dce15e (patch) | |
tree | 4819067cf6754b8b5a84f14a5e8b1614d98dc282 /src/webenginewidgets | |
parent | 531665a20c93b6b53fafdffbc3e58a923c3e3122 (diff) |
Fix crash on exit-fullscreen using context menu
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 <michael.bruning@qt.io>
Diffstat (limited to 'src/webenginewidgets')
-rw-r--r-- | src/webenginewidgets/api/qwebenginepage.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
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(); |