Oilpan: improve ScriptPromiseProperty's argument types.

Source/modules/serviceworkers/ServiceWorkerContainer.h

Index: Source/modules/serviceworkers/ServiceWorkerContainer.h
diff --git a/Source/modules/serviceworkers/ServiceWorkerContainer.h b/Source/modules/serviceworkers/ServiceWorkerContainer.h
index df0277fbf60d979fea60d330cc5f844955a4fade..c8b52e718e926d92d42c71452540a6b288235e91 100644
--- a/Source/modules/serviceworkers/ServiceWorkerContainer.h
+++ b/Source/modules/serviceworkers/ServiceWorkerContainer.h
@@ -87,7 +87,7 @@ public:
 private:
     explicit ServiceWorkerContainer(ExecutionContext*);
 
-    typedef ScriptPromiseProperty<ServiceWorkerContainer*, RefPtrWillBeMember<ServiceWorker>, RefPtrWillBeMember<ServiceWorker> > ReadyProperty;
+    typedef ScriptPromiseProperty<RefPtrWillBeMember<ServiceWorkerContainer>, RefPtrWillBeMember<ServiceWorker>, RefPtrWillBeMember<ServiceWorker> > ReadyProperty;

[Mads Ager (chromium), 2014/07/16 11:10:43]

It looks to me like this will leak through the tiny ref cycle:

ServiceWorkerContainer::m_ready -> ScriptProperty::m_host
      ^
      |                                      |
       --------------------------------------

It looks like m_ready is always set, so this needs to be RawPtrWillBeMember.

[sof, 2014/07/16 11:30:03]

Ouch, a ref pointing straight back. Thanks much, fixed.

     PassRefPtrWillBeRawPtr<ReadyProperty> createReadyProperty();
     void checkReadyChanged(PassRefPtrWillBeRawPtr<ServiceWorker> previousReadyWorker);