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

See Also:

Attachments
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
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 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

r=me

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