Bug 64910
| Summary: | [Qt] C7 (Symbian^3) crash on tapping WebView | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Damian Jansen <damian.jansen> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | ||
| Priority: | P3 | ||
| Version: | 420+ | ||
| Hardware: | S60 Hardware | ||
| OS: | Other | ||
Damian Jansen
Aleksi Hänninen has found a WebView crash on the C7 device. Unfortunately I have no C7 devices on hand to verify, but if the suggestion has merit it should be considered.
Original report is at http://bugreports.qt.nokia.com/browse/QTBUG-20427
The problem appears to be located in QGraphicsWebView::mousePressEvent - Aleksi has provided a potential fix to this issue. Quoted:
I found a workaround for the bug. In QGraphicsWebView , you shouldn't do it like this:
void QGraphicsWebView::mousePressEvent(QGraphicsSceneMouseEvent* ev)
{
if (d->page) { const bool accepted = ev->isAccepted(); d->page->event(ev); ev->setAccepted(accepted); }
if (!ev->isAccepted())
QGraphicsItem::mousePressEvent(ev);
}
Instead, you should manually create an event to be propagated into the underlying QWebPage, as follows:
const QPoint pos = ev->pos().toPoint();
QMouseEvent *event = new QMouseEvent(QEvent::MouseButtonPress, pos, ev->button(), ev->buttons(), ev->modifiers());
page()->event(event);
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
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 and remove [Qt] from the summary.
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.