Bug 40000

Summary: [Qt] QGraphicsWebView crash when calling setScale() before setUrl()
Product: WebKit Reporter: Alan Alpert <alan.alpert>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Major CC: commit-queue, hausmann, kling, laszlo.gombos
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Program that reproduces bug
none
Proposed patch none

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