Add InlinedMarkingVisitor::shouldMarkObject() predicate.

Source/platform/heap/MarkingVisitorImpl.h

 // Copyright 2014 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 MarkingVisitorImpl_h
 #define MarkingVisitorImpl_h
 
 #include "platform/heap/Heap.h"
 #include "platform/heap/ThreadState.h"
 #include "platform/heap/Visitor.h"
@@ skipping 13 matching lines @@
     {
         ASSERT(header);
         ASSERT(objectPointer);
         // Check that we are not marking objects that are outside
         // the heap by calling Heap::contains.  However we cannot
         // call Heap::contains when outside a GC and we call mark
         // when doing weakness for ephemerons.  Hence we only check
         // when called within.
         ASSERT(!ThreadState::current()->isInGC() || Heap::containedInHeapOrOrphanedPage(header));
 
-        if (toDerived()->checkSkipForObjectInTerminatingThreadHeap(objectPointer))
+        if (!toDerived()->shouldMarkObject(objectPointer))
             return;
 
         // If you hit this ASSERT, it means that there is a dangling pointer
         // from a live thread heap to a dead thread heap.  We must eliminate
         // the dangling pointer.
         // Release builds don't have the ASSERT, but it is OK because
         // release builds will crash in the following header->isMarked()
         // because all the entries of the orphaned heaps are zapped.
         ASSERT(!pageFromObject(objectPointer)->orphaned());
 
@@ skipping 41 matching lines @@
 
     inline bool isMarked(const void* objectPointer)
     {
         return GeneralHeapObjectHeader::fromPayload(objectPointer)->isMarked();
     }
 
     inline bool ensureMarked(const void* objectPointer)
     {
         if (!objectPointer)
             return false;
-        if (toDerived()->checkSkipForObjectInTerminatingThreadHeap(objectPointer))
+        if (!toDerived()->shouldMarkObject(objectPointer))
             return false;
 #if ENABLE(ASSERT)
         if (isMarked(objectPointer))
             return false;
 
         toDerived()->markNoTracing(objectPointer);
 #else
         // Inline what the above markNoTracing() call expands to,
         // so as to make sure that we do get all the benefits.
         GeneralHeapObjectHeader* header = GeneralHeapObjectHeader::fromPayload(objectPointer);
         if (header->isMarked())
             return false;
         header->mark();
 #endif
         return true;
     }
 
 #if ENABLE(ASSERT)
 #define DEFINE_ENSURE_MARKED_METHOD(Type)                                          \
     inline bool ensureMarked(const Type* objectPointer)                            \
     {                                                                              \
         if (!objectPointer)                                                        \
             return false;                                                          \
-        if (toDerived()->checkSkipForObjectInTerminatingThreadHeap(objectPointer)) \
+        if (!toDerived()->shouldMarkObject(objectPointer))                         \
             return false;                                                          \
         if (isMarked(objectPointer))                                               \
             return false;                                                          \
         toDerived()->markNoTracing(objectPointer);                                 \
         return true;                                                               \
     }
 #else
 #define DEFINE_ENSURE_MARKED_METHOD(Type)                                          \
     inline bool ensureMarked(const Type* objectPointer)                            \
     {                                                                              \
         if (!objectPointer)                                                        \
             return false;                                                          \
-        if (toDerived()->checkSkipForObjectInTerminatingThreadHeap(objectPointer)) \
+        if (!toDerived()->shouldMarkObject(objectPointer))                         \
             return false;                                                          \
         HeapObjectHeader* header = HeapObjectHeader::fromPayload(objectPointer);   \
         if (header->isMarked())                                                    \
             return false;                                                          \
         header->mark();                                                            \
         return true;                                                               \
     }
 #endif
 
 // This macro defines the necessary visitor methods for typed heaps
@@ skipping 29 matching lines @@
 private:
     static void markNoTracingCallback(Visitor* visitor, void* object)
     {
         visitor->markNoTracing(object);
     }
 };
 
 } // namespace blink
 
 #endif