[wrapper-tracing] Bring remoteplayback up to date

third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h

Index: third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h
diff --git a/third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h b/third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h
index 36d104c3df498dac1e3d320df2bf9a60c3e9e645..6872a3298684b9e1a6f3987518fe677472b700b5 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptWrappable.h
@@ -40,6 +40,15 @@
 
 namespace blink {
 
+class CORE_EXPORT TraceWrapperBase {

[Michael Lippautz, 2016/10/19 12:20:21]

We could use this super class in cases where the extra vtable pointer doesn't
matter or we cannot properly include because of component builds.

A future CL could also make it a base class for ScriptWrappable, getting rid of
the special visitor code.

[Marcel Hlopko, 2016/10/19 15:08:17]

Nit: TraceWrappersBase?

[haraken, 2016/10/19 15:31:43]

Maybe we should unify TraceWrappers and TraceWrapper into one of the two. We
have TraceWrapperMember. We have TRACE_WRAPPERS.

I'd slightly prefer TraceWrapper but anyway a follow-up CL is fine.

[Michael Lippautz, 2016/10/19 15:44:34]

Will do renaming, and inheriting changes to ScriptWrappable in a followup CL.

+  WTF_MAKE_NONCOPYABLE(TraceWrapperBase);
+
+ public:
+  TraceWrapperBase() = default;
+
+  DECLARE_VIRTUAL_TRACE_WRAPPERS(){};
+};
+
 // ScriptWrappable provides a way to map from/to C++ DOM implementation to/from
 // JavaScript object (platform object).  toV8() converts a ScriptWrappable to
 // a v8::Object and toScriptWrappable() converts a v8::Object back to