[webkit-reviews] review granted: [Bug 195623] Link prefetch not useful for top-level navigation : [Attachment 368343] Patch

[bugzilla-daemon at webkit.org]

youenn fablet <youennf at gmail.com> has granted Rob Buis <rbuis at igalia.com>'s
request for review:
Bug 195623: Link prefetch not useful for top-level navigation
https://bugs.webkit.org/show_bug.cgi?id=195623

Attachment 368343: Patch

https://bugs.webkit.org/attachment.cgi?id=368343&action=review




--- Comment #179 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 368343
  --> https://bugs.webkit.org/attachment.cgi?id=368343
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=368343&action=review

> Source/WebKit/NetworkProcess/cache/PrefetchCache.h:50
> +	   Ref<WebCore::SharedBuffer> releaseBuffer() { return
buffer.releaseNonNull(); }

I do not think we can guarantee buffer will always be null.
We might just need to WTFMove(buffer) when storing it in network cache.