Bug 83463 - [Qt][WK2] QtViewportInteractionEngine::reset() asserts after WebProcess is restarted with a page defining viewport arguments.
Summary: [Qt][WK2] QtViewportInteractionEngine::reset() asserts after WebProcess is re...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords:
: 83698 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-04-09 05:27 PDT by zalan
Modified: 2014-02-03 03:20 PST (History)
5 users (show)

See Also:


Attachments
Patch (2.96 KB, patch)
2012-04-09 05:56 PDT, zalan
kenneth: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 2012-04-09 05:27:14 PDT
1, load a page with meta viewport
2, make the webprocess crash
3, reload the page to get the webprocess restarted
UI process asserts at QtViewportInteractionEngine::reset(), because viewport update comes while the viewport is still suspended (animating, initiated by contents size changed)
Comment 1 zalan 2012-04-09 05:56:42 PDT
Created attachment 136217 [details]
Patch
Comment 2 Kenneth Rohde Christiansen 2012-04-10 02:23:22 PDT
Comment on attachment 136217 [details]
Patch

r- given comments on irc. This seems like a band-aid fix to a generic problem.
Comment 3 Jocelyn Turcotte 2012-04-11 09:53:41 PDT
*** Bug 83698 has been marked as a duplicate of this bug. ***
Comment 4 zalan 2012-04-11 10:35:13 PDT
as per bug#83698, it happens when webprocess is not crashed.
Comment 5 zalan 2012-04-11 10:40:20 PDT
discussed with Kenneth to come up with a more generic solution where delayed tasks could be added to a queue and execute them after the current UI related task (scaling, animating etc) has finished.
Comment 6 Jocelyn Turcotte 2014-02-03 03:20:34 PST
=== 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.