[webkit-reviews] review granted: [Bug 185393] Abstract logic to log console messages and send CSP violation reports into a client : [Attachment 339760] Patch

[bugzilla-daemon at webkit.org]

Brent Fulgham <bfulgham at webkit.org> has granted  review:
Bug 185393: Abstract logic to log console messages and send CSP violation
reports into a client
https://bugs.webkit.org/show_bug.cgi?id=185393

Attachment 339760: Patch

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




--- Comment #3 from Brent Fulgham <bfulgham at webkit.org> ---
Comment on attachment 339760
  --> https://bugs.webkit.org/attachment.cgi?id=339760
Patch

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

r=me

> Source/WebKit/NetworkProcess/NetworkLoadChecker.h:77
> +    WebCore::ContentSecurityPolicy* contentSecurityPolicy();

Perhaps m_contentSecurityPolicy doesn't need to be mutable if this can be
non-const.