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

Source/bindings/templates/interface.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file has been auto-generated by {{code_generator}}. DO NOT MODIFY!
 
 #ifndef {{v8_class}}_h
 #define {{v8_class}}_h
 
 {% filter conditional(conditional_string) %}
 {% for filename in header_includes %}
 #include "{{filename}}"
 {% endfor %}
 
 namespace blink {
 
 {% if has_event_constructor %}
 class Dictionary;
 {% endif %}
 {% if named_constructor %}
 class {{v8_class}}Constructor {
 public:
     static v8::Handle<v8::FunctionTemplate> domTemplate(v8::Isolate*);
     static const WrapperTypeInfo wrapperTypeInfo;
 };
 
 {% 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*);
     static {{cpp_class}}* toNative(v8::Handle<v8::Object> object)
     {
         return fromInternalPointer(object->GetAlignedPointerFromInternalField(v8DOMWrapperObjectIndex));
     }
     static {{cpp_class}}* toNativeWithTypeCheck(v8::Isolate*, v8::Handle<v8::Value>);
     static const WrapperTypeInfo wrapperTypeInfo;
     static void derefObject(void*);
     {% if has_visit_dom_wrapper %}
     static void visitDOMWrapper(void*, const v8::Persistent<v8::Object>&, v8::Isolate*);
     {% endif %}
     {% if is_active_dom_object %}
     static ActiveDOMObject* toActiveDOMObject(v8::Handle<v8::Object>);
     {% endif %}
     {% if is_event_target %}
     static EventTarget* toEventTarget(v8::Handle<v8::Object>);
     {% endif %}
     {% if interface_name == 'Window' %}
     static v8::Handle<v8::ObjectTemplate> getShadowObjectTemplate(v8::Isolate*);
     {% endif %}
     {% for method in methods %}
     {% if method.is_custom %}
     {% filter conditional(method.conditional_string) %}
     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>&);
     {% endif %}
     {% if has_custom_constructor %}
     static void constructorCustom(const v8::FunctionCallbackInfo<v8::Value>&);
     {% endif %}
     {% for attribute in attributes %}
     {% if attribute.has_custom_getter %}{# FIXME: and not attribute.implemented_by #}
     {% filter conditional(attribute.conditional_string) %}
     static void {{attribute.name}}AttributeGetterCustom(const v8::PropertyCallbackInfo<v8::Value>&);
     {% endfilter %}
     {% endif %}
     {% if attribute.has_custom_setter %}{# FIXME: and not attribute.implemented_by #}
     {% filter conditional(attribute.conditional_string) %}
     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 %}
     static void indexedPropertyGetterCustom(uint32_t, const v8::PropertyCallbackInfo<v8::Value>&);
     {% endif %}
     {% if indexed_property_setter and indexed_property_setter.is_custom %}
     static void indexedPropertySetterCustom(uint32_t, v8::Local<v8::Value>, const v8::PropertyCallbackInfo<v8::Value>&);
     {% endif %}
     {% if indexed_property_deleter and indexed_property_deleter.is_custom %}
     static void indexedPropertyDeleterCustom(uint32_t, const v8::PropertyCallbackInfo<v8::Boolean>&);
@@ skipping 192 matching lines @@
     v8SetReturnValueFast(callbackInfo, impl.get(), wrappable);
 }
 
 {% if has_event_constructor %}
 bool initialize{{cpp_class}}({{cpp_class}}Init&, const Dictionary&, ExceptionState&, const v8::FunctionCallbackInfo<v8::Value>& info, const String& = "");
 
 {% endif %}
 }
 {% endfilter %}
 #endif // {{v8_class}}_h