diff options
author | Joerg Bornemann <joerg.bornemann@qt.io> | 2017-09-08 12:17:20 +0200 |
---|---|---|
committer | Joerg Bornemann <joerg.bornemann@qt.io> | 2017-09-11 09:10:31 +0000 |
commit | e409cfca1765aad5f6cc09c0e0ebd019dd5b37e2 (patch) | |
tree | 62d51e269ebdb23cdb55e88c514e866e355c69be /src/webenginewidgets/doc/snippets/qtwebengine_qwebengineview_snippet.cpp | |
parent | f6dc11e9b589ae80a6665424b94c5e23567ea9d6 (diff) | |
download | qtwebengine-e409cfca1765aad5f6cc09c0e0ebd019dd5b37e2.tar.gz |
Remove QWebEngineViewPrivate::m_pendingContextMenuEvent
This flag was set in QWebEngineView::contextMenuEvent and checked in
QWebEnginePage::contextMenuRequested. The latter would bail out if the
flag was not set.
When the user pressed Shift-F10, the key event was received by
RenderWidgetHostViewQtDelegateWidget::event and forwarded to Chromium.
Blink's default event handler invoked the context menu, and our callback
QWebEnginePage::contextMenuRequested was called. But nothing happened,
because m_pendingContextMenuEvent has never been set.
There is no reason to have this flag. React on every context menu
request just like in the QtQuick implementation.
Now pressing Shift-F10 invokes the context menu on web pages.
Task-number: QTBUG-58306
Change-Id: I2db1b17604e5521e44613297120a964cc4e4d544
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'src/webenginewidgets/doc/snippets/qtwebengine_qwebengineview_snippet.cpp')
0 files changed, 0 insertions, 0 deletions