[Oilpan] Apply RefCountedGarbageCollectedEventTarget on AbstractWorker

Source/modules/serviceworkers/ServiceWorker.cpp

 /*
  * Copyright (C) 2013 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 83 matching lines @@
     case WebServiceWorkerStateActivated:
         return "activated";
     case WebServiceWorkerStateRedundant:
         return "redundant";
     default:
         ASSERT_NOT_REACHED();
         return nullAtom;
     }
 }
 
-PassRefPtrWillBeRawPtr<ServiceWorker> ServiceWorker::from(ExecutionContext* executionContext, WebServiceWorker* worker)
+ServiceWorker* ServiceWorker::from(ExecutionContext* executionContext, WebServiceWorker* worker)
 {
     if (!worker)

[sof, 2015/08/14 06:17:55]

nit: this null check is redundant.

[peria, 2015/08/14 07:36:14]

Done.

         return nullptr;
 
-    RefPtrWillBeRawPtr<ServiceWorker> serviceWorker = getOrCreate(executionContext, worker);
-    return serviceWorker.release();
+    return getOrCreate(executionContext, worker);
 }
 
 bool ServiceWorker::hasPendingActivity() const
 {
     if (AbstractWorker::hasPendingActivity())
         return true;
     if (m_wasStopped)
         return false;
     return m_outerWorker->state() != WebServiceWorkerStateRedundant;
 }
 
 void ServiceWorker::stop()
 {
     m_wasStopped = true;
 }
 
-PassRefPtrWillBeRawPtr<ServiceWorker> ServiceWorker::getOrCreate(ExecutionContext* executionContext, WebServiceWorker* outerWorker)
+ServiceWorker* ServiceWorker::getOrCreate(ExecutionContext* executionContext, WebServiceWorker* outerWorker)
 {
     if (!outerWorker)
         return nullptr;
 
     ServiceWorker* existingServiceWorker = static_cast<ServiceWorker*>(outerWorker->proxy());
     if (existingServiceWorker) {
         ASSERT(existingServiceWorker->executionContext() == executionContext);
         return existingServiceWorker;
     }
 
-    RefPtrWillBeRawPtr<ServiceWorker> worker = adoptRefWillBeNoop(new ServiceWorker(executionContext, adoptPtr(outerWorker)));
+    ServiceWorker* worker = new ServiceWorker(executionContext, adoptPtr(outerWorker));
     worker->suspendIfNeeded();
-    return worker.release();
+    return worker;
 }
 
 ServiceWorker::ServiceWorker(ExecutionContext* executionContext, PassOwnPtr<WebServiceWorker> worker)
     : AbstractWorker(executionContext)
     , m_outerWorker(worker)
     , m_wasStopped(false)
 {
     ASSERT(m_outerWorker);
     m_outerWorker->setProxy(this);
 }
 
 ServiceWorker::~ServiceWorker()
 {
 }
 
 } // namespace blink