Oilpan: dispose ServiceWorker (and registration) objects earlier.

Source/modules/serviceworkers/ServiceWorkerRegistration.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "ServiceWorkerRegistration.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
@@ skipping 126 matching lines @@
     registration->suspendIfNeeded();
     return registration;
 }
 
 ServiceWorkerRegistration::ServiceWorkerRegistration(ExecutionContext* executionContext, PassOwnPtr<WebServiceWorkerRegistration> outerRegistration)
     : ActiveDOMObject(executionContext)
     , m_outerRegistration(outerRegistration)
     , m_provider(0)
     , m_stopped(false)
 {
+    WILL_REGISTER_PREFINALIZER(this);
     ASSERT(m_outerRegistration);
 
     if (!executionContext)
         return;
     if (ServiceWorkerContainerClient* client = ServiceWorkerContainerClient::from(executionContext))
         m_provider = client->provider();
     m_outerRegistration->setProxy(this);
 }
 
 void ServiceWorkerRegistration::trace(Visitor* visitor)
@@ skipping 12 matching lines @@
 }
 
 void ServiceWorkerRegistration::stop()
 {
     if (m_stopped)
         return;
     m_stopped = true;
     m_outerRegistration->proxyStopped();
 }
 
+void ServiceWorkerRegistration::dispose()
+{
+    // See ServiceWorker::dispose() comment why this explicit dispose() action is needed.
+    m_outerRegistration.clear();

[haraken, 2015/01/26 01:39:42]

Can we add ASSERT(!m_outerRegistration) to ~ServiceWorkerRegistration()?

[sof, 2015/01/26 10:06:57]

Done.

+}
+
+
 } // namespace blink