[webkit-reviews] review requested: [Bug 22043] ASSERTION FAILED: this == frameLoader()->activeDocumentLoader() (../../WebCore/loader/DocumentLoader.cpp:402 void WebCore::DocumentLoader::updateLoading()) : [Attachment 24855] Do not run into an assert

[bugzilla-daemon at webkit.org]

Holger Freyther <zecke at selfish.org> has asked  for review:
Bug 22043: ASSERTION FAILED: this == frameLoader()->activeDocumentLoader()
(../../WebCore/loader/DocumentLoader.cpp:402 void
WebCore::DocumentLoader::updateLoading())
https://bugs.webkit.org/show_bug.cgi?id=22043

Attachment 24855: Do not run into an assert
https://bugs.webkit.org/attachment.cgi?id=24855&action=edit

------- Additional Comments from Holger Freyther <zecke at selfish.org>
Proposed patch to not run into the assert. From what I can see is that the CURL
backend is not really guilty. FrameLoader is currently using a RefPtr to keep
the DocumentLoader alive, so the other ports might have a similar invocation
pattern.

The question is if the call to dl->setPrimaryLoadComplete(true) can be omitted
or not.