[Webkit-unassigned] [Bug 30879] Loading HTML with a JS alert() when the DocumentLoader has been set to not defer data load results in ASSERT

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 29 12:10:13 PDT 2009


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





--- Comment #2 from Darin Adler <darin at apple.com>  2009-10-29 12:10:11 PDT ---
(In reply to comment #0)
> The problem as I understand it is that MainResourceLoader::setDefersLoading()
> does not take into account the case where the resources has already been
> loaded, so it starts a new load, either though loadNow() or through the timer.

The logic in MainResourceLoader seems a bit twisted and has changed quite a bit
since I last looked at it.

The reason m_initialRequest exists is to record that there's deferred loading
that needs to be done. It should be cleared when the resource has already been
loaded. I don't think adding a check for m_response is the right thing to do.

Adding code to clear m_initialRequest in handleDataLoadNow is probably the
right fix.

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



More information about the webkit-unassigned mailing list