Summary: | [Qt] request parameter of QQuickWebView::navigationRequested is not a registered type | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Simon Hausmann <hausmann> | ||||||
Component: | WebKit API | Assignee: | Jesus Sanchez-Palencia <jesus> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cmarcelo, gopal.1.raghavan, jesus, laszlo.gombos, menard, vestbo, webkit.review.bot, zoltan | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Simon Hausmann
2011-12-05 05:18:18 PST
Created attachment 117941 [details]
Patch
Comment on attachment 117941 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117941&action=review No test? > Source/WebKit2/UIProcess/API/qt/qwebnavigationrequest_p.h:38 > + QWebNavigationRequest(const QUrl& url, Qt::MouseButton button, Qt::KeyboardModifiers modifiers, QObject* parent = 0); What about touch? I know that is not really part of this patch, but we are exposing it so we should think about such things. Maybe it is time to look at how MS exposed touch/mouse/pointers in their IE10 api - like if there is some abstraction Comment on attachment 117941 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117941&action=review r- because of the leak. Looks good otherwise. > Source/WebKit2/UIProcess/API/qt/qwebnavigationrequest.cpp:48 > + , d(new QWebNavigationRequestPrivate(url, button, modifiers)) Ooops, the d-pointer is leaked :) Created attachment 118567 [details]
Patch
Comment on attachment 118567 [details] Patch Clearing flags on attachment: 118567 Committed r102502: <http://trac.webkit.org/changeset/102502> All reviewed patches have been landed. Closing bug. |