[webkit-reviews] review requested: [Bug 32326] Refactor some security code out of V8 bindings : [Attachment 44709] Adds newlines and a BindingSecurityBase class

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Dec 11 14:56:27 PST 2009


Charles Reis <creis at chromium.org> has asked  for review:
Bug 32326: Refactor some security code out of V8 bindings
https://bugs.webkit.org/show_bug.cgi?id=32326

Attachment 44709: Adds newlines and a BindingSecurityBase class
https://bugs.webkit.org/attachment.cgi?id=44709&action=review

------- Additional Comments from Charles Reis <creis at chromium.org>
I took Adam's suggestion to make BindingSecurityHelper into
BindingSecurityBase, and I added the requested newlines.

I'm inclined to think that this code should stay in the bindings directory, but
if WebCore proper has a better place for it, that's fine.


More information about the webkit-reviews mailing list