Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1965)

Unified Diff: Source/core/workers/InProcessWorkerBase.cpp

Issue 1257653007: Oilpan: Remove raw pointer to InProcessWorkerBase from WorkerMessagingProxy (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/workers/InProcessWorkerBase.cpp
diff --git a/Source/core/workers/InProcessWorkerBase.cpp b/Source/core/workers/InProcessWorkerBase.cpp
index 2250c7380f4d6a5a83574052d95fdfc6e4d2f13f..54ac1cc0d1dc66c16ad8042237ed81033cc56f0f 100644
--- a/Source/core/workers/InProcessWorkerBase.cpp
+++ b/Source/core/workers/InProcessWorkerBase.cpp
@@ -28,9 +28,11 @@ InProcessWorkerBase::InProcessWorkerBase(ExecutionContext* context)
InProcessWorkerBase::~InProcessWorkerBase()
{
ASSERT(isMainThread());
+#if !ENABLE(OILPAN)
if (!m_contextProxy)
return;
m_contextProxy->workerObjectDestroyed();
+#endif
}
void InProcessWorkerBase::postMessage(ExecutionContext* context, PassRefPtr<SerializedScriptValue> message, const MessagePortArray* ports, ExceptionState& exceptionState)
@@ -112,6 +114,7 @@ void InProcessWorkerBase::onFinished()
DEFINE_TRACE(InProcessWorkerBase)
{
visitor->trace(m_contentSecurityPolicy);
+ visitor->trace(m_contextProxy);
AbstractWorker::trace(visitor);
}

Powered by Google App Engine
This is Rietveld 408576698