[heap] Enforce explicit MarkingState

src/heap/objects-visiting-inl.h

 // Copyright 2012 the V8 project 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 V8_OBJECTS_VISITING_INL_H_
 #define V8_OBJECTS_VISITING_INL_H_
 
 #include "src/heap/array-buffer-tracker.h"
 #include "src/heap/mark-compact.h"
 #include "src/heap/objects-visiting.h"
@@ skipping 315 matching lines @@
 template <typename StaticVisitor>
 void StaticMarkingVisitor<StaticVisitor>::VisitWeakCell(Map* map,
                                                         HeapObject* object) {
   Heap* heap = map->GetHeap();
   WeakCell* weak_cell = reinterpret_cast<WeakCell*>(object);
   // Enqueue weak cell in linked list of encountered weak collections.
   // We can ignore weak cells with cleared values because they will always
   // contain smi zero.
   if (weak_cell->next_cleared() && !weak_cell->cleared()) {
     HeapObject* value = HeapObject::cast(weak_cell->value());
-    if (ObjectMarking::IsBlackOrGrey(value)) {
+    if (ObjectMarking::IsBlackOrGrey(value, MarkingState::Internal(value))) {
       // Weak cells with live values are directly processed here to reduce
       // the processing time of weak cells during the main GC pause.
       Object** slot = HeapObject::RawField(weak_cell, WeakCell::kValueOffset);
       map->GetHeap()->mark_compact_collector()->RecordSlot(weak_cell, slot,
                                                            *slot);
     } else {
       // If we do not know about liveness of values of weak cells, we have to
       // process them when we know the liveness of the whole transitive
       // closure.
       weak_cell->set_next(heap->encountered_weak_cells(),
@@ skipping 168 matching lines @@
 }
 
 
 template <typename StaticVisitor>
 bool StaticMarkingVisitor<StaticVisitor>::IsFlushable(Heap* heap,
                                                       JSFunction* function) {
   SharedFunctionInfo* shared_info = function->shared();
 
   // Code is either on stack, in compilation cache or referenced
   // by optimized version of function.
-  if (ObjectMarking::IsBlackOrGrey(function->code())) {
+  if (ObjectMarking::IsBlackOrGrey(function->code(),
+                                   MarkingState::Internal(function->code()))) {
     return false;
   }
 
   // We do not (yet) flush code for optimized functions.
   if (function->code() != shared_info->code()) {
     return false;
   }
 
   // Check age of optimized code.
   if (FLAG_age_code && !function->code()->IsOld()) {
     return false;
   }
 
   return IsFlushable(heap, shared_info);
 }
 
 
 template <typename StaticVisitor>
 bool StaticMarkingVisitor<StaticVisitor>::IsFlushable(
     Heap* heap, SharedFunctionInfo* shared_info) {
   // Code is either on stack, in compilation cache or referenced
   // by optimized version of function.
-  if (ObjectMarking::IsBlackOrGrey(shared_info->code())) {
+  if (ObjectMarking::IsBlackOrGrey(
+          shared_info->code(), MarkingState::Internal(shared_info->code()))) {
     return false;
   }
 
   // The function must be compiled and have the source code available,
   // to be able to recompile it in case we need the function again.
   if (!(shared_info->is_compiled() && HasSourceCode(heap, shared_info))) {
     return false;
   }
 
   // We never flush code for API functions.
@@ skipping 80 matching lines @@
   typedef FlexibleBodyVisitor<StaticVisitor, JSFunction::BodyDescriptorWeakCode,
                               void> JSFunctionWeakCodeBodyVisitor;
   JSFunctionWeakCodeBodyVisitor::Visit(map, object);
 }
 
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_OBJECTS_VISITING_INL_H_