diff options
| author | Peter Varga <pvarga@inf.u-szeged.hu> | 2020-05-08 14:05:06 +0200 |
|---|---|---|
| committer | Peter Varga <pvarga@inf.u-szeged.hu> | 2020-05-25 09:00:17 +0000 |
| commit | 4f6da986bf6aa929670e6bbf1630d51d886aa3d8 (patch) | |
| tree | 716c3a057a6f37f4e8e6686092bc81c20f5d3d8a /src/core/render_widget_host_view_qt.cpp | |
| parent | e63d27ca775429df8c3f03dc2df001e502119db1 (diff) | |
Fix scrollbar when releasing mouse button outside of window on Windows
When releasing mouse button outside of window on Windows, an extra
QEvent::Leave is sent before the QEvent::MouseButtonRelease. This
extra Leave event resets the pointer to the last interacted scrollbar
(last_scrollbar_under_mouse_) in Chromium. Leave events should not be
sent until the last mouse button is released.
Fixes: QTBUG-70437
Pick-to: 5.15
Change-Id: Ibf28b3ca0fd82feb5abf91ec851d92db0a8657f0
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
(cherry picked from commit 6eadb2f2588b240b5fcda38e1f68a49a4690eb6b)
Diffstat (limited to 'src/core/render_widget_host_view_qt.cpp')
| -rw-r--r-- | src/core/render_widget_host_view_qt.cpp | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/core/render_widget_host_view_qt.cpp b/src/core/render_widget_host_view_qt.cpp index c4d6200b7..2eeb09b2e 100644 --- a/src/core/render_widget_host_view_qt.cpp +++ b/src/core/render_widget_host_view_qt.cpp @@ -1170,8 +1170,12 @@ bool RenderWidgetHostViewQt::forwardEvent(QEvent *event) case QEvent::InputMethodQuery: handleInputMethodQueryEvent(static_cast<QInputMethodQueryEvent*>(event)); break; - case QEvent::HoverLeave: case QEvent::Leave: +#ifdef Q_OS_WIN + if (m_mouseButtonPressed > 0) + return false; +#endif + case QEvent::HoverLeave: host()->ForwardMouseEvent(WebEventFactory::toWebMouseEvent(event)); break; default: @@ -1284,6 +1288,11 @@ bool RenderWidgetHostViewQt::IsPopup() const void RenderWidgetHostViewQt::handleMouseEvent(QMouseEvent* event) { + if (event->type() == QEvent::MouseButtonPress) + m_mouseButtonPressed++; + if (event->type() == QEvent::MouseButtonRelease) + m_mouseButtonPressed--; + // Don't forward mouse events synthesized by the system, which are caused by genuine touch // events. Chromium would then process for e.g. a mouse click handler twice, once due to the // system synthesized mouse event, and another time due to a touch-to-gesture-to-mouse |