Have ScriptPromiseResolver on the Oilpan heap always.

Source/bindings/core/v8/CallbackPromiseAdapter.h

 /*
  * 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 58 matching lines @@
 //
 // Note:
 // - This class does not manage its own lifetime. In this example that ownership
 //   of the WebCallbacks instance is being passed in and it is up to the callee
 //   to free the WebCallbacks instance.
 // - onSuccess and onError take ownership of the given WebType instance.
 template<typename S, typename T>
 class CallbackPromiseAdapter final : public WebCallbacks<typename S::WebType*, typename T::WebType*> {
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 public:
-    explicit CallbackPromiseAdapter(PassRefPtrWillBeRawPtr<ScriptPromiseResolver> resolver)
+    explicit CallbackPromiseAdapter(ScriptPromiseResolver* resolver)
         : m_resolver(resolver)
     {
         ASSERT(m_resolver);
     }
     ~CallbackPromiseAdapter() override { }
 
     // Takes ownership of |result|.
     void onSuccess(typename S::WebType* result) override
     {
         OwnPtr<typename S::WebType> ownPtr = adoptPtr(result);
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->resolve(S::take(m_resolver.get(), ownPtr.release()));
     }
 
     // Takes ownership of |error|.
     void onError(typename T::WebType* error) override
     {
         OwnPtr<typename T::WebType> ownPtr = adoptPtr(error);
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->reject(T::take(m_resolver.get(), ownPtr.release()));
     }
 
 private:
-    RefPtrWillBePersistent<ScriptPromiseResolver> m_resolver;
+    Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 template<typename T>
 class CallbackPromiseAdapter<void, T> final : public WebCallbacks<void, typename T::WebType*> {
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 public:
-    explicit CallbackPromiseAdapter(PassRefPtrWillBeRawPtr<ScriptPromiseResolver> resolver)
+    explicit CallbackPromiseAdapter(ScriptPromiseResolver* resolver)
         : m_resolver(resolver)
     {
         ASSERT(m_resolver);
     }
     ~CallbackPromiseAdapter() override { }
 
     void onSuccess() override
     {
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->resolve();
     }
 
     // Takes ownership of |error|.
     void onError(typename T::WebType* error) override
     {
         OwnPtr<typename T::WebType> ownPtr = adoptPtr(error);
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->reject(T::take(m_resolver.get(), ownPtr.release()));
     }
 
 private:
-    RefPtrWillBePersistent<ScriptPromiseResolver> m_resolver;
+    Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 template<typename S>
 class CallbackPromiseAdapter<S, void> final : public WebCallbacks<typename S::WebType*, void> {
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 public:
-    explicit CallbackPromiseAdapter(PassRefPtrWillBeRawPtr<ScriptPromiseResolver> resolver)
+    explicit CallbackPromiseAdapter(ScriptPromiseResolver* resolver)
         : m_resolver(resolver)
     {
         ASSERT(m_resolver);
     }
     ~CallbackPromiseAdapter() override { }
 
     // Takes ownership of |result|.
     void onSuccess(typename S::WebType* result) override
     {
         OwnPtr<typename S::WebType> ownPtr = adoptPtr(result);
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->resolve(S::take(m_resolver.get(), ownPtr.release()));
     }
 
     void onError() override
     {
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->reject();
     }
 
 private:
-    RefPtrWillBePersistent<ScriptPromiseResolver> m_resolver;
+    Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 template<typename T>
 class CallbackPromiseAdapter<bool, T> final : public WebCallbacks<bool*, typename T::WebType*> {
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 public:
-    explicit CallbackPromiseAdapter(PassRefPtrWillBeRawPtr<ScriptPromiseResolver> resolver)
+    explicit CallbackPromiseAdapter(ScriptPromiseResolver* resolver)
         : m_resolver(resolver)
     {
         ASSERT(m_resolver);
     }
     ~CallbackPromiseAdapter() override { }
 
     // TODO(nhiroki): onSuccess should take ownership of a bool object for
     // consistency. (http://crbug.com/493531)
     void onSuccess(bool* result) override
     {
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->resolve(*result);
     }
 
     // Takes ownership of |error|.
     void onError(typename T::WebType* error) override
     {
         OwnPtr<typename T::WebType> ownPtr = adoptPtr(error);
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->reject(T::take(m_resolver.get(), ownPtr.release()));
     }
 
 private:
-    RefPtrWillBePersistent<ScriptPromiseResolver> m_resolver;
+    Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 template<>
 class CallbackPromiseAdapter<void, void> final : public WebCallbacks<void, void> {
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 public:
-    explicit CallbackPromiseAdapter(PassRefPtrWillBeRawPtr<ScriptPromiseResolver> resolver)
+    explicit CallbackPromiseAdapter(ScriptPromiseResolver* resolver)
         : m_resolver(resolver)
     {
         ASSERT(m_resolver);
     }
     ~CallbackPromiseAdapter() override { }
 
     void onSuccess() override
     {
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->resolve();
     }
 
     void onError() override
     {
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->reject();
     }
 
 private:
-    RefPtrWillBePersistent<ScriptPromiseResolver> m_resolver;
+    Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 template<>
 class CallbackPromiseAdapter<bool, void> final : public WebCallbacks<bool*, void> {
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 public:
-    explicit CallbackPromiseAdapter(PassRefPtrWillBeRawPtr<ScriptPromiseResolver> resolver)
+    explicit CallbackPromiseAdapter(ScriptPromiseResolver* resolver)
         : m_resolver(resolver)
     {
         ASSERT(m_resolver);
     }
     ~CallbackPromiseAdapter() override { }
 
     // TODO(nhiroki): onSuccess should take ownership of a bool object for
     // consistency. (http://crbug.com/493531)
     void onSuccess(bool* result) override
     {
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->resolve(*result);
     }
 
     void onError() override
     {
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
         m_resolver->reject();
     }
 
 private:
-    RefPtrWillBePersistent<ScriptPromiseResolver> m_resolver;
+    Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 } // namespace blink
 
 #endif