Oilpan: Replace most of RefPtrs for Event objects with oilpan's transition types

Source/bindings/v8/WorkerScriptController.h

Index: Source/bindings/v8/WorkerScriptController.h
diff --git a/Source/bindings/v8/WorkerScriptController.h b/Source/bindings/v8/WorkerScriptController.h
index 5b46cf9ed7c84d6496075c5ef5d121018d0c3ad3..b2fb9f9dbe52193efdcf8ce83b0ce35ac8907e2e 100644
--- a/Source/bindings/v8/WorkerScriptController.h
+++ b/Source/bindings/v8/WorkerScriptController.h
@@ -68,9 +68,9 @@ namespace WebCore {
 
         WorkerGlobalScope& workerGlobalScope() { return m_workerGlobalScope; }
 
-        void evaluate(const ScriptSourceCode&, RefPtr<ErrorEvent>* = 0);
+        void evaluate(const ScriptSourceCode&, RefPtrWillBeRawPtr<ErrorEvent>* = 0);
 
-        void rethrowExceptionFromImportedScript(PassRefPtr<ErrorEvent>);
+        void rethrowExceptionFromImportedScript(PassRefPtrWillBeRawPtr<ErrorEvent>);
 
         // Async request to terminate a future JS execution. Eventually causes termination
         // exception raised during JS execution, if the worker thread happens to run JS.
@@ -111,7 +111,7 @@ namespace WebCore {
         bool m_executionForbidden;
         bool m_executionScheduledToTerminate;
         mutable Mutex m_scheduledTerminationMutex;
-        RefPtr<ErrorEvent> m_errorEventFromImportedScript;
+        RefPtrWillBeRawPtr<ErrorEvent> m_errorEventFromImportedScript;

[sof, 2014/03/28 12:27:18]

Why is a raw pointer safe here? If it is, could you add an explanatory comment?

[haraken, 2014/03/28 14:02:14]

Good catch. This should be a Persistent. Fixed.

         OwnPtr<V8IsolateInterruptor> m_interruptor;
     };