[webkit-reviews] review denied: [Bug 130534] Move cross-port Source/WebCore/page/ code to std::unique_ptr : [Attachment 227791] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Apr 16 11:48:42 PDT 2014
Brent Fulgham <bfulgham at webkit.org> has denied Zan Dobersek
<zandobersek at gmail.com>'s request for review:
Bug 130534: Move cross-port Source/WebCore/page/ code to std::unique_ptr
https://bugs.webkit.org/show_bug.cgi?id=130534
Attachment 227791: Patch
https://bugs.webkit.org/attachment.cgi?id=227791&action=review
------- Additional Comments from Brent Fulgham <bfulgham at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=227791&action=review
This looks good, but unfortunately seemed to choke in the EWS system for Mac.
Could you rebaseline and upload again so we can confirm it's okay?
> Source/WebCore/page/ContentSecurityPolicy.cpp:919
> + std::unique_ptr<CSPDirectiveList> directives(new
CSPDirectiveList(policy, type));
Why not auto directives = std::make_unique<CSPDirectiveList>(policy, type)?
More information about the webkit-reviews
mailing list