[wrapper-tracing] Add tracing to WebGLRenderingContextBase and friends

third_party/WebKit/Source/modules/webgl/WebGLFramebuffer.cpp

Index: third_party/WebKit/Source/modules/webgl/WebGLFramebuffer.cpp
diff --git a/third_party/WebKit/Source/modules/webgl/WebGLFramebuffer.cpp b/third_party/WebKit/Source/modules/webgl/WebGLFramebuffer.cpp
index 06e3fdf3f54c49b07ef2904622f26b48ec0c26af..9dbf9903e8343c071c805c320d85f6cce7ec86d8 100644
--- a/third_party/WebKit/Source/modules/webgl/WebGLFramebuffer.cpp
+++ b/third_party/WebKit/Source/modules/webgl/WebGLFramebuffer.cpp
@@ -545,4 +545,11 @@ DEFINE_TRACE(WebGLFramebuffer) {
   WebGLContextObject::trace(visitor);
 }
 
+DEFINE_TRACE_WRAPPERS(WebGLFramebuffer) {
+  for (const auto& attachment : m_attachments) {
+    visitor->traceWrappers(attachment.value->object());

[Michael Lippautz, 2016/10/06 17:54:54]

I mirrored the object grouping behavior here by directly retaining
WebGLAttachment::object(). Should we also keep the intermediate WebGLAttachment?

[Ken Russell (switch to Gerrit), 2016/10/06 19:05:14]

There's no JavaScript wrapper for it, so that's not necessary.

[Michael Lippautz, 2016/10/06 19:58:13]

Yep, it's not necessary. 

I guess my question is whether we should try to avoid shortcuts and follow the
object graph (in this case, make WebGLAttachment tracable and trace the object()
in there), or just mirror what object grouping did.

I am fine with both.

[Ken Russell (switch to Gerrit), 2016/10/07 03:00:55]

Let's just mirror what object grouping did -- i.e., don't add a traceWrappers to
WebGLAttachment. That class is an implementation detail and it makes sense to me
that traceWrappers should really only be added to ScriptWrappables.

[haraken, 2016/10/07 04:48:49]

We've already added a bunch of traceWrappers to non-ScriptWrappables, because we
sometimes need to tell DOM-side reachability to V8.

That being said, I don't have any strong opinion about how
WebGLFramebuffer::traceWrappers should be written. The current CL looks fine.

[Ken Russell (switch to Gerrit), 2016/10/07 04:58:52]

OK. I don't have a strong opinion on this.

+  }
+  WebGLContextObject::traceWrappers(visitor);
+}
+
 }  // namespace blink