[Webkit-unassigned] [Bug 39670] [Qt] WebView::setUrl crashes after Qt4.7

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=39670





--- Comment #15 from Jan Erik Hanssen <jhanssen at gmail.com>  2010-12-06 15:55:56 PST ---
(In reply to comment #14)
> (In reply to comment #12)
> > (In reply to comment #9)
> > > I can't reproduce this crash against Qt 4.6 and ToT WebKit or against qt-git and ToT WebKit. There's no longer a WebCore::CachedResource::setDocLoader in WebCore. Maybe it has been fixed by a drive-by?
> > 
> > Just had time to look at this again and still seeing the problem as of r73340 with Qt 4.7.1.
> 
> I succeeded reproducing it with qtwebkit-2.0 (the version released with qt-4.7) and with trunk (r73392), but not on the qtwebkit-2.1 branch... I'll investigate more later... maybe it's a race... any ideas?
> 
> Could you prepare a patch for trunk inclusion? Backporting your current patch for 2.0 was simple, but the right thing to do is to fix this on trunk (with proper review) and then cherry-pick/backport it to stable branches.

There is a race, yes. If the subresource (the css file) has been processed by the loader before the second setUrl() is called then the problem will not occur. My test case attempts to avoid this by loading the resource from a php script that calls sleep() to ensure that the subresource is still being loaded at this point.

I'll upload a new patch for trunk inclusion.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.