Remove WorkerScriptLoaderClient and inheritances

Source/core/workers/InProcessWorkerBase.h

Index: Source/core/workers/InProcessWorkerBase.h
diff --git a/Source/core/workers/InProcessWorkerBase.h b/Source/core/workers/InProcessWorkerBase.h
index 18c3171a3fadff21f29200aa6816b2d78c1d2abf..a92ef81623d11a7dd645e035f83c5d6cb3175456 100644
--- a/Source/core/workers/InProcessWorkerBase.h
+++ b/Source/core/workers/InProcessWorkerBase.h
@@ -11,7 +11,6 @@
 #include "core/events/EventListener.h"
 #include "core/events/EventTarget.h"
 #include "core/workers/AbstractWorker.h"
-#include "core/workers/WorkerScriptLoaderClient.h"
 #include "platform/heap/Handle.h"
 #include "wtf/Forward.h"
 #include "wtf/PassRefPtr.h"
@@ -25,10 +24,11 @@ class ExecutionContext;
 class WorkerGlobalScopeProxy;
 class WorkerScriptLoader;
 
-// Base class for workers that operate in the same process as the document that creates them.
-class CORE_EXPORT InProcessWorkerBase : public AbstractWorker, private WorkerScriptLoaderClient {
+// Base class for workers that operate in the same process as the document that
+// creates them.

[Mike West, 2015/06/24 07:18:33]

Nit: There's no 80-column limit in Blink, so there's no need to change this
comment.

[Takashi Toyoshima, 2015/06/24 08:53:45]

Yeah, you are right. But sometime top directory owner suggests me to wrap
comments in 80-columns. That must not be mandatory, I think. But I also prefer
80-column comments if it does not make things worse.

+class CORE_EXPORT InProcessWorkerBase : public AbstractWorker {
 public:
-    virtual ~InProcessWorkerBase();
+    ~InProcessWorkerBase() override;
 
     void postMessage(ExecutionContext*, PassRefPtr<SerializedScriptValue> message, const MessagePortArray*, ExceptionState&);
     void terminate();
@@ -52,11 +52,11 @@ protected:
     virtual WorkerGlobalScopeProxy* createWorkerGlobalScopeProxy(ExecutionContext*) = 0;
 
 private:
-    // WorkerScriptLoaderClient callbacks
-    void didReceiveResponse(unsigned long identifier, const ResourceResponse&) override;
-    void notifyFinished() override;
+    // Callbacks for m_scriptLoader.
+    void onResponse();
+    void onFinished();
 
-    RefPtr<WorkerScriptLoader> m_scriptLoader;
+    OwnPtr<WorkerScriptLoader> m_scriptLoader;
     RefPtr<ContentSecurityPolicy> m_contentSecurityPolicy;
     WorkerGlobalScopeProxy* m_contextProxy; // The proxy outlives the worker to perform thread shutdown.
 };