Per the discussion in bug 93239, (1) Rename V8ConfigureDOMAttributesAndMethods to V8DOMConfiguration. (2) Introduce V8DOMConfiguration class and make batchConfigureXXX() methods static.
Created attachment 157617 [details] Patch
Created attachment 157622 [details] Patch
Comment on attachment 157622 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=157622&action=review > Source/WebCore/bindings/v8/V8DOMConfiguration.cpp:37 > + v8::Handle<v8::ObjectTemplate> proto, I would clean up all these names too. e.g., proto -> prototype > Source/WebCore/bindings/v8/V8DOMConfiguration.cpp:67 > +v8::Local<v8::Signature> V8DOMConfiguration::configureTemplate(v8::Persistent<v8::FunctionTemplate> desc, desc -> functionDescriptor > Source/WebCore/bindings/v8/V8DOMConfiguration.cpp:68 > + const char *interfaceName, "const char *" -> "const char*" > Source/WebCore/bindings/v8/V8DOMConfiguration.cpp:70 > + int fieldCount, int -> size_t ? > Source/WebCore/bindings/v8/V8DOMConfiguration.h:53 > + bool onProto; onProto -> onPrototype > Source/WebCore/bindings/v8/V8DOMConfiguration.h:63 > + v8::Handle<ObjectOrTemplate> proto, proto -> prototype > Source/WebCore/bindings/v8/V8DOMConfiguration.h:65 > + { Bad indent
Created attachment 157726 [details] patch for landing
Comment on attachment 157726 [details] patch for landing Clearing flags on attachment: 157726 Committed r125295: <http://trac.webkit.org/changeset/125295>