[webkit-reviews] review granted: [Bug 94433] Add call stacks to Content Security Policy checks when relevant. : [Attachment 161022] Chromium vs Everything Else.

[bugzilla-daemon at webkit.org]

Yury Semikhatsky <yurys at chromium.org> has granted Mike West
<mkwst at chromium.org>'s request for review:
Bug 94433: Add call stacks to Content Security Policy checks when relevant.
https://bugs.webkit.org/show_bug.cgi?id=94433

Attachment 161022: Chromium vs Everything Else.
https://bugs.webkit.org/attachment.cgi?id=161022&action=review

------- Additional Comments from Yury Semikhatsky <yurys at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=161022&action=review


r=me

> Source/WebCore/ChangeLog:48
> +	       stack trace just before logging it, and only pass it to

Nice.

> LayoutTests/http/tests/inspector/resources/csp-test.js:4
> +    var messageCount = 0;

You don't need this counter. Just drop the second argument in the call to
InspectorTest.addConsoleSniffer above.