[webkit-reviews] review granted: [Bug 131580] [iOS] Move QuickLookHandle from ResourceLoader to WebResourceLoader : [Attachment 229196] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Andy Estes <aestes at apple.com>'s
request for review:
Bug 131580: [iOS] Move QuickLookHandle from ResourceLoader to WebResourceLoader
https://bugs.webkit.org/show_bug.cgi?id=131580

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

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=229196&action=review


> Source/WebKit2/WebProcess/Network/WebResourceLoader.h:98
> +    OwnPtr<WebCore::QuickLookHandle> m_quickLookHandle;

Should use std::unique_ptr instead of OwnPtr, but I suppose that requires
changing the QuickLookHandle class itself.