StackFrameDepth should be managed per ThreadHeap

third_party/WebKit/Source/platform/heap/TraceTraits.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef TraceTraits_h
 #define TraceTraits_h
 
 #include "platform/heap/GCInfo.h"
 #include "platform/heap/Heap.h"
 #include "platform/heap/InlinedGlobalMarkingVisitor.h"
@@ skipping 52 matching lines @@
         // If the trait allows it, invoke the trace callback right here on the
         // not-yet-marked object.
         if (TraceEagerlyTrait<T>::value) {
             // Protect against too deep trace call chains, and the
             // unbounded system stack usage they can bring about.
             //
             // Assert against deep stacks so as to flush them out,
             // but test and appropriately handle them should they occur
             // in release builds.
             //
-            ASSERT(StackFrameDepth::isAcceptableStackUse());
-            if (LIKELY(StackFrameDepth::isSafeToRecurse())) {
+            DCHECK(visitor->heap().stackFrameDepth().isAcceptableStackUse());
+            if (LIKELY(visitor->heap().stackFrameDepth().isSafeToRecurse())) {

[haraken, 2016/09/02 09:07:57]

I hope this won't regress performance.

Moving StackFrameDepthScope to Visitor might help the compiler to generate more
optimized code.

                 if (visitor->ensureMarked(t)) {
                     TraceTrait<T>::trace(visitor, const_cast<T*>(t));
                 }
                 return;
             }
         }
         visitor->mark(const_cast<T*>(t), &TraceTrait<T>::trace);
     }
 };
 
@@ skipping 522 matching lines @@
         // since iterating over the hash table backing will find the whole
         // chain.
         visitor->markNoTracing(node);
         return false;
     }
 };
 
 } // namespace WTF
 
 #endif