Store raw pointers in ScriptWrappableVisitor markind deque

third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.h

Index: third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.h
diff --git a/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.h b/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.h
index be5c990df420426034dcfefcddbf052b577cf5fc..5632480e96012239b68f4c233c5d67e7cd523aee 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.h
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.h
@@ -20,6 +20,51 @@ namespace blink {
 class HeapObjectHeader;
 template<typename T> class Member;
 
+class WrapperMarkingData {
+public:
+    WrapperMarkingData(
+        void (*traceWrappersCallback)(const WrapperVisitor*, const void*),
+        HeapObjectHeader* (*heapObjectHeaderCallback)(const void*),
+        const void* object)
+        : m_traceWrappersCallback(traceWrappersCallback)
+        , m_heapObjectHeaderCallback(heapObjectHeaderCallback)
+        , m_object(object)
+    {
+        DCHECK(m_traceWrappersCallback);
+        DCHECK(m_heapObjectHeaderCallback);
+        DCHECK(m_object);
+    }
+
+
+    inline void traceWrappers(WrapperVisitor* visitor)
+    {
+        if (m_object) {
+            m_traceWrappersCallback(visitor, m_object);
+        }
+    }
+
+    inline bool shouldBeInvalidated()
+    {
+        return m_object && !heapObjectHeader()->isMarked();
+    }
+
+    inline void invalidate()
+    {
+        m_object = nullptr;
+    }
+
+private:
+    inline const HeapObjectHeader* heapObjectHeader()
+    {
+        DCHECK(m_object);
+        return m_heapObjectHeaderCallback(m_object);
+    }
+
+    void (*m_traceWrappersCallback)(const WrapperVisitor*, const void*);
+    HeapObjectHeader* (*m_heapObjectHeaderCallback)(const void*);
+    const void* m_object;
+};
+
 /**
  * ScriptWrappableVisitor is able to trace through the script wrappable
  * references. It is used during V8 garbage collection.  When this visitor is
@@ -39,11 +84,6 @@ public:
      */
     static void invalidateDeadObjectsInMarkingDeque(v8::Isolate*);
     /**
-     * Mark wrappers in all worlds for the given script wrappable as alive in
-     * V8.
-     */
-    static void markWrappersInAllWorlds(const ScriptWrappable*, v8::Isolate*);
-    /**
      * Mark given wrapper as alive in V8.
      */
     template <typename T>
@@ -73,16 +113,31 @@ public:
     WRAPPER_VISITOR_SPECIAL_CLASSES(DECLARE_DISPATCH_TRACE_WRAPPERS);
 
 #undef DECLARE_DISPATCH_TRACE_WRAPPERS
-    virtual void dispatchTraceWrappers(const void*) const {}
+    void dispatchTraceWrappers(const void*) const override {}
 
     void traceWrappers(const ScopedPersistent<v8::Value>*) const override;
 
     void invalidateDeadObjectsInMarkingDeque();
 
-private:
+    void pushToMarkingDeque(
+        void (*traceWrappersCallback)(const WrapperVisitor*, const void*),
+        HeapObjectHeader* (*heapObjectHeaderCallback)(const void*),
+        const void* object) const override
+    {
+        m_markingDeque.append(WrapperMarkingData(
+            traceWrappersCallback,
+            heapObjectHeaderCallback,
+            object));
+    }
+
     bool markWrapperHeader(HeapObjectHeader*) const;
-    bool markWrapperHeader(const ScriptWrappable*) const override;
-    bool markWrapperHeader(const void* garbageCollected) const override;
+    /**
+     * Mark wrappers in all worlds for the given script wrappable as alive in
+     * V8.
+     */
+    void markWrappersInAllWorlds(const ScriptWrappable*) const override;
+    void markWrappersInAllWorlds(const void*) const override {}
+private:
     bool m_tracingInProgress = false;
     void performCleanup();
     /**
@@ -92,7 +147,7 @@ private:
      *     * for the oilpan gc, this deque is part of the root set, so objects
      *       in the deque will not die
      */
-    mutable WTF::Deque<const ScriptWrappable*> m_markingDeque;
+    mutable WTF::Deque<WrapperMarkingData> m_markingDeque;
     /**
      * Collection of headers we need to unmark after the tracing finished. We
      * assume it is safe to hold on to the headers because: