[webkit-reviews] review granted: [Bug 32326] Refactor some security code out of V8 bindings : [Attachment 44696] Update BindingSecurity class to use templates

[bugzilla-daemon at webkit.org]

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

Attachment 44696: Update BindingSecurity class to use templates
https://bugs.webkit.org/attachment.cgi?id=44696&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
This looks like a great first step.  I've noted a few formatting issues below. 
There are a couple of things that I'm not sure how they'll scale up when we
convert more of the bindings, but they're fine for now.  We can revisit them if
the code isn't growing the right direction.

+ template <class Binding> class BindingSecurity {

I think we'd prefer

+ template <class Binding>
+ class BindingSecurity {

(meaning a line break)

+ class BindingSecurityHelper

I'm not entirely sold on this class.  We might need to see more examples to see
what the best general pattern is for this kind of thing.

+ typedef BindingSecurity<V8Binding> V8BindingSecurity;

I'm also not sure how this will scale.	It's fine here though.

+ template <> class State<V8Binding> : public State<GenericBinding> {

We'd also like a line break here.