[webkit-reviews] review granted: [Bug 69728] CSP should log unrecognized directives to the console : [Attachment 110311] Patch

[bugzilla-daemon at webkit.org]

Sam Weinig <sam at webkit.org> has granted Adam Barth <abarth at webkit.org>'s
request for review:
Bug 69728: CSP should log unrecognized directives to the console
https://bugs.webkit.org/show_bug.cgi?id=69728

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

------- Additional Comments from Sam Weinig <sam at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=110311&action=review


> Source/WebCore/page/ContentSecurityPolicy.h:80
> +    void reportUnrecognizedDirective(const String& name) const;

I am not thrilled with using the word report here, as it has a subtle
implication that it will have the same semantics as reportViolation, sending
the report to the server.