Bug 94199 – Pop-up menu is not displayed when right-clicking on a chart

• Component: WebKit Qt
• Status: RESOLVED INVALID

[Jouni K]

Created attachment 158734 [details]
Screenshot of the problem

Steps to reproduce the bug:
- launch the Qt web browser demo or any browser based on QtWebkit
- go to http://si.galaxysemi.com 
- log with user : [e-mail] password : digia55 
- select dashboard 'DemoDashboard' in the upper right corner combo 
- click on 'MaverickLotReport' tab (if it doesn't display, refresh the page)
- On the 'Hold Report by Work Week' chart, right-click on a bar. A popup menu should open, but nothing happens.

The bug is in all versions of QWebkit (from at least 2.0 to 2.2)

[Pierre Rossi]

preliminary investigation showed the subframe in which the context menu event handler is installed does not have focus, leading to doing hit testing on the wrong document in sendContextMenuEvent, which in turn means we end up dispatching the contextMenuEvent to the wrong node.

[Jocelyn Turcotte]

=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.

[Clement IGONET]

A workaround:
--- qt-everywhere-enterprise-src-5.2.1/qtwebkit/Source/WebCore/page/qt/EventHandlerQt.cpp	2013-08-25 22:05:46.000000000 +0200
+++ qt-everywhere-enterprise-new-5.2.1/qtwebkit/Source/WebCore/page/qt/EventHandlerQt.cpp	2013-11-04 10:44:25.871325348 +0100
@@ -101,6 +101,8 @@
 
 bool EventHandler::passMousePressEventToSubframe(MouseEventWithHitTestResults& mev, Frame* subframe)
 {
+    if (subframe->page())
+        subframe->page()->focusController()->setFocusedFrame(subframe);
     subframe->eventHandler()->handleMousePressEvent(mev.event());
     return true;
 }