We no longer want to passively allow DocumentLoader::detachFromFrame() to be called twice as it never makes sense to for it to be called twice and indicates a bug that we want to fix. A release assert in DocumentLoader::cancelPolicyCheckIfNeeded() (added in r187558) called by DocumentLoader::detachFromFrame() enforces this invariant. We should remove the unnecessary null check of DocumentLoader::m_frameand the comment that explains this null check from DocumentLoader::detachFromFrame().
(In reply to Daniel Bates from comment #0) > [...] > the unnecessary null check of DocumentLoader::m_frameand the comment that > explains this null check from DocumentLoader::detachFromFrame(). *DocumentLoader::m_frame and
Created attachment 308915 [details] Patch
Comment on attachment 308915 [details] Patch Clearing flags on attachment: 308915 Committed r216119: <http://trac.webkit.org/changeset/216119>
All reviewed patches have been landed. Closing bug.