Make ScriptStreamer and dependents Oilpan friendly.

Source/bindings/core/v8/PageScriptDebugServer.h

 /*
  * Copyright (c) 2011 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 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef PageScriptDebugServer_h
 #define PageScriptDebugServer_h
 
+#include "bindings/core/v8/Nullable.h"
 #include "bindings/core/v8/ScriptDebugServer.h"
-#include "bindings/core/v8/ScriptPreprocessor.h"
+#include "bindings/core/v8/ScriptSourceCode.h"
 #include <v8.h>
 
 namespace blink {
 
 class Page;
 class ScriptPreprocessor;
-class ScriptSourceCode;
 
 class PageScriptDebugServer final : public ScriptDebugServer {
     WTF_MAKE_NONCOPYABLE(PageScriptDebugServer);
 public:
     static PageScriptDebugServer& shared();
 
+    ~PageScriptDebugServer() override;
+    void trace(Visitor*) override;
+
     static void setMainThreadIsolate(v8::Isolate*);
 
     void addListener(ScriptDebugListener*, Page*);
     void removeListener(ScriptDebugListener*, Page*);
 
     static void interruptAndRun(PassOwnPtr<Task>);
 
     class ClientMessageLoop {
     public:
         virtual ~ClientMessageLoop() { }
         virtual void run(Page*) = 0;
         virtual void quitNow() = 0;
     };
     void setClientMessageLoop(PassOwnPtr<ClientMessageLoop>);
 
     void compileScript(ScriptState*, const String& expression, const String& sourceURL, String* scriptId, String* exceptionDetailsText, int* lineNumber, int* columnNumber, RefPtrWillBeRawPtr<ScriptCallStack>* stackTrace) override;
     void clearCompiledScripts() override;
     void runScript(ScriptState*, const String& scriptId, ScriptValue* result, bool* wasThrown, String* exceptionDetailsText, int* lineNumber, int* columnNumber, RefPtrWillBeRawPtr<ScriptCallStack>* stackTrace) override;
     void setPreprocessorSource(const String&) override;
     void preprocessBeforeCompile(const v8::Debug::EventDetails&) override;
-    PassOwnPtr<ScriptSourceCode> preprocess(LocalFrame*, const ScriptSourceCode&) override;
+    Nullable<ScriptSourceCode> preprocess(LocalFrame*, const ScriptSourceCode&) override;

[haraken, 2015/01/22 02:11:54]

Using Nullable<> looks a bit tricky. At the moment, Nullable<> is just intended
to be used to represent nullability of IDL values and not intended to be used to
represent nullable C++ pointers (In Blink, we use a raw pointer for a nullable
C++ pointer and use a reference for a non-nullable C++ pointer).

Shall we change m_preprocessorSourceCode to ScriptSourceCode* and add
GC_PLUGIN_IGNORE? Or a better approach would be to make ScriptSourceCode
GarbageCollected instead of ALLOW_ONLY_INLINE_ALLOCATION. Then we can simply use
Member<ScriptSourceCode>. Given that ScriptSourceCode is instantiated only per
script, the overhead of making it on-heap allocated would be negligible.

     String preprocessEventListener(LocalFrame*, const String& source, const String& url, const String& functionName) override;
     void clearPreprocessor() override;
 
     void muteWarningsAndDeprecations() override;
     void unmuteWarningsAndDeprecations() override;
 
 private:
     PageScriptDebugServer();
-    ~PageScriptDebugServer() override;
 
     ScriptDebugListener* getDebugListenerForContext(v8::Handle<v8::Context>) override;
     void runMessageLoopOnPause(v8::Handle<v8::Context>) override;
     void quitMessageLoopOnPause() override;
 
-    typedef HashMap<Page*, ScriptDebugListener*> ListenersMap;
+    using ListenersMap = WillBeHeapHashMap<RawPtrWillBeMember<Page>, ScriptDebugListener*>;
     ListenersMap m_listenersMap;
     OwnPtr<ClientMessageLoop> m_clientMessageLoop;
-    Page* m_pausedPage;
+    RawPtrWillBeMember<Page> m_pausedPage;
     HashMap<String, String> m_compiledScriptURLs;
 
-    OwnPtr<ScriptSourceCode> m_preprocessorSourceCode;
+    Nullable<ScriptSourceCode> m_preprocessorSourceCode;
     OwnPtr<ScriptPreprocessor> m_scriptPreprocessor;
     bool canPreprocess(LocalFrame*);
     static v8::Isolate* s_mainThreadIsolate;
 };
 
 } // namespace blink
 
 
 #endif // PageScriptDebugServer_h