Make NodeFilter a legacy callback interface.

third_party/WebKit/Source/bindings/templates/callback_interface.h.tmpl

 {% filter format_blink_cpp_source_code %}
 
 {% include 'copyright_block.txt' %}
 #ifndef {{v8_class}}_h
 #define {{v8_class}}_h
 
 {% for filename in header_includes %}
 #include "{{filename}}"
 {% endfor %}
 
 namespace blink {
 
+{% if is_legacy %}
+{# TODO(tkent): Merge two class declaration templates.
+   Even if this is a legacy callback interface, we should generate a class
+   to call a v8::Function or a method of a v8::Object. We don't do so now due
+   to crbug.com/630986.
+#}
+class {{v8_class}} {

[Yuki, 2017/04/27 12:18:29]

nit: final?

[tkent, 2017/04/28 06:13:36]

Done.

+  STATIC_ONLY({{v8_class}});
+ public:
+  static v8::Local<v8::FunctionTemplate> DomTemplate(v8::Isolate*, const DOMWrapperWorld&);

[Yuki, 2017/04/27 12:18:29]

nit: {{exported}}
due to the snapshot project.

[peria, 2017/04/27 13:14:49]

It is OK to keep callback interfaces' DomTemplate() not exported.

+  static const WrapperTypeInfo wrapperTypeInfo;

[Yuki, 2017/04/27 12:18:29]

nit: {{exported}}
due to the snapshot project.

[peria, 2017/04/27 13:14:49]

Yes, please put it.

[tkent, 2017/04/28 06:13:37]

Done.

+  static const int internalFieldCount = kV8DefaultWrapperInternalFieldCount;
+};
+
+{% else %} {# is_legacy #}
+
 class {{v8_class}} final : public {{cpp_class}} {
  public:
   static {{v8_class}}* Create(v8::Local<v8::Function> callback, ScriptState* scriptState) {
     return new {{v8_class}}(callback, scriptState);
   }
 
   ~{{v8_class}}() override;
 
   DECLARE_VIRTUAL_TRACE();
 
 {% for method in methods %}
   {{method.cpp_type}} {{method.name}}({{method.argument_declarations | join(', ')}}) override;
 {% endfor %}
 
  private:
   {{exported}}{{v8_class}}(v8::Local<v8::Function>, ScriptState*);
 
   ScopedPersistent<v8::Function> m_callback;
   RefPtr<ScriptState> m_scriptState;
 };
+{% endif %} {# is_legacy #}
 
 }
 #endif  // {{v8_class}}_h
 
 {% endfilter %}{# format_blink_cpp_source_code #}