[webkit-reviews] review granted: [Bug 69294] Move ContentSecurityPolicy to the ScriptExecutionContext to prepare it for working with XHR and workers : [Attachment 109535] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Sam Weinig <sam at webkit.org>'s request
for review:
Bug 69294: Move ContentSecurityPolicy to the ScriptExecutionContext to prepare
it for working with XHR and workers
https://bugs.webkit.org/show_bug.cgi?id=69294

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

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


> Source/WebCore/dom/ScriptExecutionContext.h:178
>	   // Note: It is dangerous to change the security origin of a script
context
>	   //	    that already contains content.
>	   void setSecurityOrigin(PassRefPtr<SecurityOrigin>);
> +	   void setContentSecurityPolicy(PassRefPtr<ContentSecurityPolicy>);

Paragraphing here is a little strange since the comment above is about the
security origin, not the content security policy.