[webkit-reviews] review granted: [Bug 88470] MHTMLArchives are leaked when a MHTML file is loaded : [Attachment 146147] Patch

[bugzilla-daemon at webkit.org]

Adam Barth <abarth at webkit.org> has granted Jay Civelli
<jcivelli at chromium.org>'s request for review:
Bug 88470: MHTMLArchives are leaked when a MHTML file is loaded
https://bugs.webkit.org/show_bug.cgi?id=88470

Attachment 146147: Patch
https://bugs.webkit.org/attachment.cgi?id=146147&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=146147&action=review


> Source/WebCore/loader/archive/Archive.cpp:46
> +    for (Vector<RefPtr<Archive> >::iterator it = m_subframeArchives.begin();
it != m_subframeArchives.end(); ++it)

This loop should have { } around its body.