Chromium Code Reviews

Unified Diff: third_party/WebKit/Source/bindings/templates/interface.h

Issue 1873323002: Have bindings layer assume and insist that all interface types are GCed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: third_party/WebKit/Source/bindings/templates/interface.h
diff --git a/third_party/WebKit/Source/bindings/templates/interface.h b/third_party/WebKit/Source/bindings/templates/interface.h
index bfc651b182261d5147bf9932db0ec15437a6f2ec..b1ee90bd880b95b370745fd5869456717e40f306 100644
--- a/third_party/WebKit/Source/bindings/templates/interface.h
+++ b/third_party/WebKit/Source/bindings/templates/interface.h
@@ -59,14 +59,10 @@ public:
{% else %}
{{exported}}static const WrapperTypeInfo wrapperTypeInfo;
{% endif %}
- static void refObject(ScriptWrappable*);
- static void derefObject(ScriptWrappable*);
template<typename VisitorDispatcher>
static void trace(VisitorDispatcher visitor, ScriptWrappable* scriptWrappable)
{
- {% if gc_type == 'GarbageCollectedObject' %}
visitor->trace(scriptWrappable->toImpl<{{cpp_class}}>());
- {% endif %}
}
{% if has_visit_dom_wrapper %}
static void visitDOMWrapper(v8::Isolate*, ScriptWrappable*, const v8::Persistent<v8::Object>&);

Powered by Google App Engine