diff options
author | Kirill Burtsev <kirill.burtsev@qt.io> | 2019-07-03 18:36:40 +0200 |
---|---|---|
committer | Kirill Burtsev <kirill.burtsev@qt.io> | 2019-07-08 10:06:24 +0000 |
commit | d3915634c392f8a350e5ef112f740d4d353daaac (patch) | |
tree | 5dc03be606216ddd361eb6273c474773be021d44 /src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp | |
parent | 9bf361eddea701a65ea2a26f3d598ec4d4a6e22c (diff) |
Fully setup replaced QMouseDblClickEvent with source and flags
Not fully preserving properties causes event to not be considered
synthesized by system and be forwarded into chromium. Receiving one
single press event and no release event confuses input event_handler
and triggers infinite loop and renderer process crash for pages with
multiple nested iframes.
Fixes: QTBUG-62106
Change-Id: Iabb8bff78fc3475923b4aa3209b720453a5b6ce5
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
Diffstat (limited to 'src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp')
-rw-r--r-- | src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp b/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp index 27adef919..18f1e97d0 100644 --- a/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp +++ b/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp @@ -471,7 +471,7 @@ bool RenderWidgetHostViewQtDelegateWidget::event(QEvent *event) // where we can simply ignore the DblClick event. QMouseEvent *dblClick = static_cast<QMouseEvent *>(event); QMouseEvent press(QEvent::MouseButtonPress, dblClick->localPos(), dblClick->windowPos(), dblClick->screenPos(), - dblClick->button(), dblClick->buttons(), dblClick->modifiers()); + dblClick->button(), dblClick->buttons(), dblClick->modifiers(), dblClick->source()); press.setTimestamp(dblClick->timestamp()); handled = m_client->forwardEvent(&press); } else |