Cleanup: Remove WeakPtr usage in WebSharedWorkerImpl

Source/web/WebSharedWorkerImpl.h

 /*
  * Copyright (C) 2009 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 80 matching lines @@
     virtual int64_t serviceWorkerID(WebDataSource&) override;
 
     // WebDevToolsAgentClient overrides.
     virtual void sendProtocolMessage(int callId, const WebString&, const WebString&) override;
     virtual void resumeStartup() override;
 
     // WebSharedWorker methods:
     virtual void startWorkerContext(const WebURL&, const WebString& name, const WebString& contentSecurityPolicy, WebContentSecurityPolicyType) override;
     virtual void connect(WebMessagePortChannel*) override;
     virtual void terminateWorkerContext() override;
-    virtual void clientDestroyed() override;
 
     virtual void pauseWorkerContextOnStart() override;
     virtual void attachDevTools(const WebString& hostId) override;
     virtual void reattachDevTools(const WebString& hostId, const WebString& savedState) override;
     virtual void detachDevTools() override;
     virtual void dispatchDevToolsMessage(const WebString&) override;
 
 private:
     class Loader;
 
     virtual ~WebSharedWorkerImpl();
 
-    WebSharedWorkerClient* client() { return m_client->get(); }
-
     void setWorkerThread(PassRefPtr<WorkerThread> thread) { m_workerThread = thread; }
     WorkerThread* workerThread() { return m_workerThread.get(); }
 
     // Shuts down the worker thread.
     void terminateWorkerThread();
 
     // Creates the shadow loader used for worker network requests.
     void initializeLoader();
 
     void loadShadowPage();
@@ skipping 17 matching lines @@
     WebLocalFrameImpl* m_mainFrame;
     bool m_askedToTerminate;
 
     // This one is bound to and used only on the main thread.
     OwnPtr<WebServiceWorkerNetworkProvider> m_networkProvider;
 
     OwnPtr<WorkerInspectorProxy> m_workerInspectorProxy;
 
     RefPtr<WorkerThread> m_workerThread;
 
-    // This one's initialized and bound to the main thread.
-    RefPtr<WeakReference<WebSharedWorkerClient>> m_client;
-
-    // Usually WeakPtr is created by WeakPtrFactory exposed by Client
-    // class itself, but here it's implemented by Chrome so we create
-    // our own WeakPtr.
-    WeakPtr<WebSharedWorkerClient> m_clientWeakPtr;
+    WebSharedWorkerClient* m_client;
 
     bool m_pauseWorkerContextOnStart;
     bool m_isPausedOnStart;
 
     // Kept around only while main script loading is ongoing.
     OwnPtr<Loader> m_mainScriptLoader;
 
     RefPtr<WorkerLoaderProxy> m_loaderProxy;
 
     WebURL m_url;
     WebString m_name;
 };
 
 } // namespace blink
 
 #endif // WebSharedWorkerImpl_h