V8 minor GC should not ignore hasPendingActivity

third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp

Index: third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp b/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
index 7fb4f48543c9c0dbd414bc9fa0eda454aadfe0e7..e2529db33f3ed209855e6e3844b64472310f71a2 100644
--- a/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
@@ -122,10 +122,6 @@ public:
 
         v8::Local<v8::Object> wrapper = v8::Local<v8::Object>::New(m_isolate, v8::Persistent<v8::Object>::Cast(*value));
         ASSERT(V8DOMWrapper::hasInternalFieldsSet(wrapper));
-        const WrapperTypeInfo* type = toWrapperTypeInfo(wrapper);

[sof, 2016/02/25 13:08:13]

You still need this, I think, but need to add marking to it (like there used to
be.) i.e., the hasPendingActivity() check below is only for image elements.

[haraken, 2016/02/25 13:58:37]

Ah, nice catch. HTMLMediaElement needs to be marked as well.

I'll update the CL.

-        if (type != npObjectTypeInfo() && toScriptWrappable(wrapper)->hasPendingActivity())
-            return;
-
         if (classId == WrapperTypeInfo::NodeClassId) {
             ASSERT(V8Node::hasInstance(wrapper, m_isolate));
             Node* node = V8Node::toImpl(wrapper);