Oilpan: Don't limit the number of wrappers collected in minor GC cycles

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

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 103 matching lines @@
         if (classId != WrapperTypeInfo::NodeClassId)
             return;
 
         // To make minor GC cycle time bounded, we limit the number of wrappers handled
         // by each minor GC cycle to 10000. This value was selected so that the minor
         // GC cycle time is bounded to 20 ms in a case where the new space size
         // is 16 MB and it is full of wrappers (which is almost the worst case).
         // Practically speaking, as far as I crawled real web applications,
         // the number of wrappers handled by each minor GC cycle is at most 3000.
         // So this limit is mainly for pathological micro benchmarks.
+        //
+        // In Oilpan, we don't limit the number of wrappers to collect as many
+        // wrappers in minor GC cycles as possible. This may increase the pause
+        // time of a minor GC, but if we give up collecting wrappers in a minor
+        // GC, it will instead end up with increasing the cost of subsequent
+        // Oilpan's GCs. Thus it will be better to collect as many wrappers as
+        // possible for minimizing the value of max(a pause time of a minor GC,
+        // a pause time of Oilpan's GC).
+#if !ENABLE(OILPAN)
         const unsigned wrappersHandledByEachMinorGC = 10000;
         if (m_nodesInNewSpace.size() >= wrappersHandledByEachMinorGC)
             return;
+#endif
 
         v8::Local<v8::Object> wrapper = v8::Local<v8::Object>::New(m_isolate, v8::Persistent<v8::Object>::Cast(*value));
         ASSERT(V8DOMWrapper::hasInternalFieldsSet(wrapper));
         ASSERT(V8Node::hasInstance(wrapper, m_isolate));
         Node* node = V8Node::toImpl(wrapper);
         // A minor DOM GC can handle only node wrappers in the main world.
         // Note that node->wrapper().IsEmpty() returns true for nodes that
         // do not have wrappers in the main world.
         if (node->containsWrapper()) {
             const WrapperTypeInfo* type = toWrapperTypeInfo(wrapper);
@@ skipping 396 matching lines @@
     Visitor* m_visitor;
 };
 
 void V8GCController::traceDOMWrappers(v8::Isolate* isolate, Visitor* visitor)
 {
     DOMWrapperTracer tracer(visitor);
     v8::V8::VisitHandlesWithClassIds(isolate, &tracer);
 }
 
 } // namespace blink