Bug 71452 - [Qt][WK2] Failing assertion with desktop webview when touch mocking in MinBrowser
Summary: [Qt][WK2] Failing assertion with desktop webview when touch mocking in MinBro...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Hausmann
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-03 03:03 PDT by Simon Hausmann
Modified: 2011-11-03 05:10 PDT (History)
2 users (show)

See Also:

Attachments
[Qt][WK2] Failing assertion with desktop webview when touch mocking in MinBrowser (2.21 KB, patch)
2011-11-03 03:08 PDT, Simon Hausmann 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Hausmann 2011-11-03 03:03:44 PDT 
[Qt][WK2] Failing assertion with desktop webview when touch mocking in MinBrowser
Comment 1 Simon Hausmann 2011-11-03 03:06:21 PDT 
When using the MiniBrowser with the desktop view, the touch mocking ends up sending a touch event to the desktop view. That causes a ASSERT_NOT_REACHED() in QtDesktopWebPageProxy::doneWithTouchEvent().

This is due to QDesktopWebView re-implementing touchEvent and the fix for bug #71310 introducing simple touch event forwarding for WTR.
Comment 2 Simon Hausmann 2011-11-03 03:08:15 PDT 
Created attachment 113451 [details]
[Qt][WK2] Failing assertion with desktop webview when touch mocking in MinBrowser
Comment 3 Andreas Kling 2011-11-03 05:08:10 PDT 
Comment on attachment 113451 [details]
[Qt][WK2] Failing assertion with desktop webview when touch mocking in MinBrowser

rs=me
Comment 4 Simon Hausmann 2011-11-03 05:09:58 PDT 
Comment on attachment 113451 [details]
[Qt][WK2] Failing assertion with desktop webview when touch mocking in MinBrowser

Clearing flags on attachment: 113451

Committed r99170: <http://trac.webkit.org/changeset/99170>
Comment 5 Simon Hausmann 2011-11-03 05:10:05 PDT 
All reviewed patches have been landed.  Closing bug.