Oilpan: Ship Oilpan for serviceworkers/

Source/modules/serviceworkers/CacheStorage.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 "modules/serviceworkers/CacheStorage.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
@@ skipping 83 matching lines @@
         m_resolver->reject(Cache::domExceptionForCacheError(*reason));
         m_resolver.clear();
     }
 
 private:
     RefPtr<ScriptPromiseResolver> m_resolver;
 };
 
 }
 
-PassRefPtrWillBeRawPtr<CacheStorage> CacheStorage::create(WebServiceWorkerCacheStorage* webCacheStorage)
+CacheStorage* CacheStorage::create(WebServiceWorkerCacheStorage* webCacheStorage)

[dominicc (has gone to gerrit), 2014/08/26 14:01:34]

Now that this is a simple "new" can we remove this helper method?

[haraken, 2014/09/03 07:49:10]

As far as I know, Blink prefers having a create method in general even if it's
simple. So let me keep that.

 {
-    return adoptRefWillBeNoop(new CacheStorage(webCacheStorage));
+    return new CacheStorage(webCacheStorage);
 }
 
 ScriptPromise CacheStorage::get(ScriptState* scriptState, const String& cacheName)
 {
     RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
     const ScriptPromise promise = resolver->promise();
 
     if (m_webCacheStorage)
         m_webCacheStorage->dispatchGet(new CacheStorageWithCacheCallbacks(resolver), cacheName);
     else
@@ skipping 53 matching lines @@
 
     return promise;
 }
 
 CacheStorage::CacheStorage(WebServiceWorkerCacheStorage* webCacheStorage) : m_webCacheStorage(webCacheStorage)
 {
     ScriptWrappable::init(this);
 }
 
 } // namespace blink