Disables MinorGC from collecting object wrappers

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 c3b4cc8eb022f32066119c36b08e2de4fc941a77..b77f24b5578e064f72f99c3f7f88374511da2600 100644
--- a/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
@@ -143,6 +143,13 @@ public:
                 v8::Persistent<v8::Object>::Cast(*value).MarkActive();
                 return;
             }
+        }  else {
+            // If there are any references to this object mark it as active.
+            // Current MinorGC pass does not handle references.
+            if (type->visitDOMWrapperFunction) {

[haraken, 2015/11/24 12:59:08]

Unfortunately this check is not correct. type->visitDOMWrapperFunction means
that this wrapper has a reference to some wrapper, not that this object is
referenced by some wrapper.

[mythria, 2015/11/24 14:21:44]

Thank you for your review. Reverted this change. 

+                v8::Persistent<v8::Object>::Cast(*value).MarkActive();

[mythria, 2015/11/24 11:50:19]

Would this check be sufficient to filter out all object wrappers that could
contain a reference? If it is not then I will revert it back to the earlier
code, where we ignore all object wrappers in minor GC.

+                return;
+            }
         }
     }