Ensure Document::responseReceived and clearResource are called on the main thread This might help debugging rdar://problem/36256931
Created attachment 335974 [details] Patch
Comment on attachment 335974 [details] Patch Clearing flags on attachment: 335974 Committed r229690: <https://trac.webkit.org/changeset/229690>
All reviewed patches have been landed. Closing bug.
<rdar://problem/38566063>
Comment on attachment 335974 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335974&action=review > Source/WebCore/loader/DocumentLoader.cpp:704 > + ASSERT(isMainThread()); I don't think that this qualifies as "ensure". "Try to find cases when not", maybe.
(In reply to Alexey Proskuryakov from comment #5) > Comment on attachment 335974 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=335974&action=review > > > Source/WebCore/loader/DocumentLoader.cpp:704 > > + ASSERT(isMainThread()); > > I don't think that this qualifies as "ensure". "Try to find cases when not", > maybe. ... or are we ensuring by crashing otherwise?
That's only in debug mode.