Bug 40000 - [Qt] QGraphicsWebView crash when calling setScale() before setUrl()
Summary: [Qt] QGraphicsWebView crash when calling setScale() before setUrl()
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Major
Assignee: Nobody
Keywords: Qt, QtTriaged
Depends on:
Reported: 2010-06-01 08:24 PDT by Alan Alpert
Modified: 2011-04-19 05:15 PDT (History)
4 users (show)

See Also:

Program that reproduces bug (2.17 KB, application/x-compressed-tar)
2010-06-01 08:24 PDT, Alan Alpert
no flags Details
Proposed patch (2.12 KB, patch)
2010-06-28 00:13 PDT, Andreas Kling
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alan Alpert 2010-06-01 08:24:50 PDT
Created attachment 57546 [details]
Program that reproduces bug

QGraphicsWebView crashes if you set the contentsScale before the URL/HTML.

Attached is an example program which crashes on startup for this reason. Built against the Webkit included in Qt 4.7 (at June 1, commit 283e899c7d57c003facc5ed8aef8e5dacf2eb945, I guessed a version from that field's options because bugzilla gets an internal server error without it).
Comment 1 Laszlo Gombos 2010-06-08 11:16:04 PDT
Thanks. If you have stack trace for the crash that would be useful - otherwise we should be able to collect that.
Comment 2 Andreas Kling 2010-06-28 00:13:25 PDT
Created attachment 59876 [details]
Proposed patch
Comment 3 Simon Hausmann 2010-06-28 00:18:55 PDT
Comment on attachment 59876 [details]
Proposed patch


Nice bug number :)
Comment 4 WebKit Commit Bot 2010-06-28 00:36:07 PDT
Comment on attachment 59876 [details]
Proposed patch

Clearing flags on attachment: 59876

Committed r61984: <http://trac.webkit.org/changeset/61984>
Comment 5 WebKit Commit Bot 2010-06-28 00:36:11 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Simon Hausmann 2010-06-28 01:58:11 PDT
Revision r61984 cherry-picked into qtwebkit-2.0 with commit 19f5af1665ba3f2d99a0f68a23e10d3920271aad