Oilpan: Remove raw pointer to InProcessWorkerBase from WorkerMessagingProxy

Source/core/workers/WorkerMessagingProxy.h

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 34 matching lines @@
 class ExecutionContext;
 class InProcessWorkerBase;
 class WorkerClients;
 class WorkerInspectorProxy;
 
 class CORE_EXPORT WorkerMessagingProxy
     : public WorkerGlobalScopeProxy
     , private WorkerLoaderProxyProvider {
     WTF_MAKE_NONCOPYABLE(WorkerMessagingProxy);
 public:
+    DECLARE_VIRTUAL_TRACE();
+
     // Implementations of WorkerGlobalScopeProxy.
     // (Only use these methods in the worker object thread.)
     void startWorkerGlobalScope(const KURL& scriptURL, const String& userAgent, const String& sourceCode, WorkerThreadStartMode) override;
     void terminateWorkerGlobalScope() override;
     void postMessageToWorkerGlobalScope(PassRefPtr<SerializedScriptValue>, PassOwnPtr<MessagePortChannelArray>) override;
     bool hasPendingActivity() const override;
     void workerObjectDestroyed() override;
 
     // These methods come from worker context thread via WorkerObjectProxy
     // and are called on the worker object thread (e.g. main thread).
     void postMessageToWorkerObject(PassRefPtr<SerializedScriptValue>, PassOwnPtr<MessagePortChannelArray>);
     void reportException(const String& errorMessage, int lineNumber, int columnNumber, const String& sourceURL, int exceptionId);
     void reportConsoleMessage(MessageSource, MessageLevel, const String& message, int lineNumber, const String& sourceURL);
     void postMessageToPageInspector(const String&);
     void postWorkerConsoleAgentEnabled();
     WorkerInspectorProxy* workerInspectorProxy();
     void confirmMessageFromWorkerObject(bool hasPendingActivity);
     void reportPendingActivity(bool hasPendingActivity);
     void workerGlobalScopeClosed();
     void workerThreadTerminated();
 
     void workerThreadCreated(PassRefPtr<WorkerThread>);
 
+    void clearWeakMembers(Visitor*);
+
 protected:
     WorkerMessagingProxy(InProcessWorkerBase*, PassOwnPtrWillBeRawPtr<WorkerClients>);
     ~WorkerMessagingProxy() override;
 
     virtual PassRefPtr<WorkerThread> createWorkerThread(double originTime) = 0;
 
     PassRefPtr<WorkerLoaderProxy> loaderProxy() { return m_loaderProxy; }
     WorkerObjectProxy& workerObjectProxy() { return *m_workerObjectProxy.get(); }
 
 private:
     void workerObjectDestroyedInternal();
     void terminateInternally();
 
     // WorkerLoaderProxyProvider
     // These methods are called on different threads to schedule loading
     // requests and to send callbacks back to WorkerGlobalScope.
     void postTaskToLoader(PassOwnPtr<ExecutionContextTask>) override;
     bool postTaskToWorkerGlobalScope(PassOwnPtr<ExecutionContextTask>) override;
 
-    RefPtrWillBePersistent<ExecutionContext> m_executionContext;
+    RefPtrWillBeMember<ExecutionContext> m_executionContext;
     OwnPtr<WorkerObjectProxy> m_workerObjectProxy;
-    InProcessWorkerBase* m_workerObject;
+    RawPtrWillBeWeakMember<InProcessWorkerBase> m_workerObject;
     bool m_mayBeDestroyed;
     RefPtr<WorkerThread> m_workerThread;
 
     unsigned m_unconfirmedMessageCount; // Unconfirmed messages from worker object to worker thread.
     bool m_workerThreadHadPendingActivity; // The latest confirmation from worker thread reported that it was still active.
 
     bool m_askedToTerminate;
 
     Vector<OwnPtr<ExecutionContextTask>> m_queuedEarlyTasks; // Tasks are queued here until there's a thread object created.
     OwnPtr<WorkerInspectorProxy> m_workerInspectorProxy;
 
-    OwnPtrWillBePersistent<WorkerClients> m_workerClients;
+    OwnPtrWillBeMember<WorkerClients> m_workerClients;
 
     RefPtr<WorkerLoaderProxy> m_loaderProxy;
+
+#if ENABLE(OILPAN)
+    GC_PLUGIN_IGNORE("366546")
+    OwnPtr<Persistent<WorkerMessagingProxy>> m_keepAlive;

[haraken, 2015/07/30 11:46:31]

Once https://codereview.chromium.org/1233173002/ is landed, we can use the
SelfKeepAlive handle.

+#endif
 };
 
 } // namespace blink
 
 #endif // WorkerMessagingProxy_h