Refactor Script(Loader|Runner): don't access Resources all over the place...

Source/bindings/core/v8/ScriptStreamer.h

Index: Source/bindings/core/v8/ScriptStreamer.h
diff --git a/Source/bindings/core/v8/ScriptStreamer.h b/Source/bindings/core/v8/ScriptStreamer.h
index 78fa1cbc7e2a387bc835345646d88e676ebe6160..5dea593eb1ebede3051c3bf762e39a2f50496cdd 100644
--- a/Source/bindings/core/v8/ScriptStreamer.h
+++ b/Source/bindings/core/v8/ScriptStreamer.h
@@ -66,8 +66,8 @@ public:
     void addClient(ScriptResourceClient* client)
     {
         ASSERT(!m_client);
-        ASSERT(!isFinished());

[haraken, 2014/10/15 15:09:41]

Can we keep this ASSERT?

We want to avoid a situation where addClient is reentered.

[marja, 2014/10/16 15:19:03]

No, since ScriptLoader::prepareScript calls this for scripts which are already
finished and wants to get the notification immediately.

This is also consistent with Resource::addClient.

         m_client = client;
+        notifyFinishedToClient();
     }
 
     void removeClient(ScriptResourceClient* client)