Oilpan: Move WorkerClients to the managed heap and trace its supplements.

Source/core/workers/WorkerGlobalScope.cpp

Index: Source/core/workers/WorkerGlobalScope.cpp
diff --git a/Source/core/workers/WorkerGlobalScope.cpp b/Source/core/workers/WorkerGlobalScope.cpp
index 7e2cdf428eca98127aaf9548624ee194389e5055..0e54ec87710e182c599b50f9e88debd06a78c166 100644
--- a/Source/core/workers/WorkerGlobalScope.cpp
+++ b/Source/core/workers/WorkerGlobalScope.cpp
@@ -74,7 +74,7 @@ public:
     virtual bool isCleanupTask() const { return true; }
 };
 
-WorkerGlobalScope::WorkerGlobalScope(const KURL& url, const String& userAgent, WorkerThread* thread, double timeOrigin, PassOwnPtr<WorkerClients> workerClients)
+WorkerGlobalScope::WorkerGlobalScope(const KURL& url, const String& userAgent, WorkerThread* thread, double timeOrigin, PassOwnPtrWillBeRawPtr<WorkerClients> workerClients)
     : m_url(url)
     , m_userAgent(userAgent)
     , m_script(adoptPtr(new WorkerScriptController(*this)))
@@ -312,9 +312,8 @@ void WorkerGlobalScope::trace(Visitor* visitor)
     visitor->trace(m_console);
     visitor->trace(m_location);
     visitor->trace(m_navigator);
-#if ENABLE(OILPAN)

[haraken, 2014/04/09 09:04:39]

Is it OK to remove the flag?

[zerny-chromium, 2014/04/09 09:31:47]

Yes, but I need to make it WillBeHeap...

Thanks for spotting!

+    visitor->trace(m_workerClients);
     HeapSupplementable<WorkerGlobalScope>::trace(visitor);
-#endif
 }
 
 } // namespace WebCore