Bug 90875 - [Qt][WK2] ASSERT: "!m_viewportItem->isMoving()" in QtViewportHandler::flickMoveEnded()
Summary: [Qt][WK2] ASSERT: "!m_viewportItem->isMoving()" in QtViewportHandler::flickMo...
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: Andras Becsi
URL:
Keywords:
Depends on: 90960 90961
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-10 06:05 PDT by Andras Becsi
Modified: 2012-07-12 03:05 PDT (History)
8 users (show)

See Also:


Attachments
Patch (3.86 KB, patch)
2012-07-10 06:07 PDT, Andras Becsi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andras Becsi 2012-07-10 06:05:46 PDT
[Qt][WK2] ASSERT: "!m_viewportItem->isMoving()" in QtViewportHandler::flickMoveEnded()
Comment 1 Andras Becsi 2012-07-10 06:07:39 PDT
Created attachment 151449 [details]
Patch
Comment 2 Kenneth Rohde Christiansen 2012-07-10 22:13:45 PDT
Comment on attachment 151449 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=151449&action=review

> Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp:1661
> +    // touch events too, thus to hinder the canvas from sending synthesized

remove "too"?

> Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp:1663
> +    // childMouseEventFilter and filter incomming touch events as well.

incoming*
Comment 3 Andras Becsi 2012-07-11 02:43:13 PDT
Committed r122318: <http://trac.webkit.org/changeset/122318>
Comment 4 Andras Becsi 2012-07-11 02:43:50 PDT
Comment on attachment 151449 [details]
Patch

Clearing flags from attachment.
Comment 5 Csaba Osztrogonác 2012-07-11 03:14:46 PDT
(In reply to comment #3)
> Committed r122318: <http://trac.webkit.org/changeset/122318>

It made 11 tests fail. Could you check it, please? Here is the new bug report: https://bugs.webkit.org/show_bug.cgi?id=90960
Comment 6 WebKit Review Bot 2012-07-11 03:38:51 PDT
Re-opened since this is blocked by 90961
Comment 7 Andras Becsi 2012-07-12 03:05:04 PDT
Committed r122438: <http://trac.webkit.org/changeset/122438>