Rename V8X::fooMethodImplementedInPrivateScript to V8X::PrivateScript::fooMethod

Source/bindings/templates/interface.h

Index: Source/bindings/templates/interface.h
diff --git a/Source/bindings/templates/interface.h b/Source/bindings/templates/interface.h
index 17b1f971efde2c8dc08e4a7faffa96c6764b23e9..a16b12a2b94bc9638a1ce2f8d8afc18e5d147d1c 100644
--- a/Source/bindings/templates/interface.h
+++ b/Source/bindings/templates/interface.h
@@ -27,6 +27,23 @@ public:
 {% endif %}
 class {{v8_class}} {
 public:
+    class PrivateScript {

[bashi, 2014/08/08 06:58:51]

nit: We might want to avoid generating PrivateScript class if there is no
implemented_in_private_script attribute/method.

+    public:
+        {% for method in methods %}

[bashi, 2014/08/08 06:58:51]

nit:
{% for method in methods if method.is_implemented_in_private_script %}

then, you can remove the following if block.

[haraken, 2014/08/08 07:01:47]

Done.

+        {% if method.is_implemented_in_private_script %}
+        static bool {{method.name}}Method({{method.argument_declarations_for_private_script | join(', ')}});
+        {% endif %}
+        {% endfor %}
+        {% for attribute in attributes %}

[bashi, 2014/08/08 06:58:51]

nit: same as above

[haraken, 2014/08/08 07:01:47]

Done.

+        {% if attribute.is_implemented_in_private_script %}
+        static bool {{attribute.name}}AttributeGetter(LocalFrame* frame, {{cpp_class}}* holderImpl, {{attribute.cpp_type}}* result);
+        {% if not attribute.is_read_only or attribute.put_forwards %}
+        static bool {{attribute.name}}AttributeSetter(LocalFrame* frame, {{cpp_class}}* holderImpl, {{attribute.argument_cpp_type}} cppValue);
+        {% endif %}
+        {% endif %}
+        {% endfor %}
+    };
+
     static bool hasInstance(v8::Handle<v8::Value>, v8::Isolate*);
     static v8::Handle<v8::Object> findInstanceInPrototypeChain(v8::Handle<v8::Value>, v8::Isolate*);
     static v8::Handle<v8::FunctionTemplate> domTemplate(v8::Isolate*);
@@ -55,9 +72,6 @@ public:
     static void {{method.name}}MethodCustom(const v8::FunctionCallbackInfo<v8::Value>&);
     {% endfilter %}
     {% endif %}
-    {% if method.is_implemented_in_private_script %}
-    static bool {{method.name}}MethodImplementedInPrivateScript({{method.argument_declarations_for_private_script | join(', ')}});
-    {% endif %}
     {% endfor %}
     {% if constructors or has_custom_constructor or has_event_constructor %}
     static void constructorCallback(const v8::FunctionCallbackInfo<v8::Value>&);
@@ -76,12 +90,6 @@ public:
     static void {{attribute.name}}AttributeSetterCustom(v8::Local<v8::Value>, const v8::PropertyCallbackInfo<void>&);
     {% endfilter %}
     {% endif %}
-    {% if attribute.is_implemented_in_private_script %}
-    static bool {{attribute.name}}AttributeGetterImplementedInPrivateScript(LocalFrame* frame, {{cpp_class}}* holderImpl, {{attribute.cpp_type}}* result);
-    {% if not attribute.is_read_only or attribute.put_forwards %}
-    static bool {{attribute.name}}AttributeSetterImplementedInPrivateScript(LocalFrame* frame, {{cpp_class}}* holderImpl, {{attribute.argument_cpp_type}} cppValue);
-    {% endif %}
-    {% endif %}
     {% endfor %}
     {# Custom special operations #}
     {% if indexed_property_getter and indexed_property_getter.is_custom %}