(WONT COMMIT) Pass a boPass a bool object allocated on heap to CallbackPromiseAdapter::onSuccess (2)

Source/bindings/core/v8/CallbackPromiseAdapter.h

Index: Source/bindings/core/v8/CallbackPromiseAdapter.h
diff --git a/Source/bindings/core/v8/CallbackPromiseAdapter.h b/Source/bindings/core/v8/CallbackPromiseAdapter.h
index 666bccc485d7bb2689fa90a0413408c824ddb41b..205e04c06d308b5dda1dce74a6645662a242fe02 100644
--- a/Source/bindings/core/v8/CallbackPromiseAdapter.h
+++ b/Source/bindings/core/v8/CallbackPromiseAdapter.h
@@ -36,6 +36,10 @@
 
 namespace blink {
 
+// TODO(nhiroki): Remove ifdef hack after Chromium-side patch is landed.
+// http://crbug.com/493531
+#define CRBUG_493531
+
 // This class provides an easy way to convert from a Script-exposed
 // class (i.e. a class that has a toV8() overload) that uses Promises
 // to a WebKit API class that uses WebCallbacks. You can define
@@ -182,13 +186,13 @@ public:
     }
     virtual ~CallbackPromiseAdapter() { }
 
-    // TODO(nhiroki): onSuccess should take ownership of a bool object for
-    // consistency. (http://crbug.com/493531)
+    // Takes ownership of |result|.
     virtual void onSuccess(bool* result) override
     {
+        OwnPtr<bool> ownPtr = adoptPtr(result);
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
-        m_resolver->resolve(*result);
+        m_resolver->resolve(*ownPtr);
     }
 
     // Takes ownership of |error|.
@@ -244,13 +248,13 @@ public:
     }
     virtual ~CallbackPromiseAdapter() { }
 
-    // TODO(nhiroki): onSuccess should take ownership of a bool object for
-    // consistency. (http://crbug.com/493531)
+    // Takes ownership of |result|.
     virtual void onSuccess(bool* result) override
     {
+        OwnPtr<bool> ownPtr = adoptPtr(result);
         if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
             return;
-        m_resolver->resolve(*result);
+        m_resolver->resolve(*ownPtr);

[haraken, 2015/07/03 05:57:49]

This will end up with calling resolveOrReject(OwnPtr<bool>, ...), which will
call toV8(OwnPtr<bool>, ...). This seems not quite right to me.

hirano-san?

[yhirano, 2015/07/03 06:04:12]

This statement passes |*ownPtr| of type |bool|, so it looks good. Am I missing
anything?

[haraken, 2015/07/03 06:09:06]

ah, OK. Then what's the merit of using an OwnPtr? It looks a bit strange to use
OwnPtr<bool>.

[nhiroki, 2015/07/03 06:59:39]

Sorry, I'm not really sure why you feel this a bit strange... which solution are
you thinking?

(1) manually deleting an object,
(2) passing an object w/o ownership as the original implementation, or
(3) passing an object by value

Re (1): I'd prefer a scoped pointer to avoid accidental leak.
Re (2): This looks a bit awkward to me because other types are passed with
ownership.
Re (3): This should be feasible, but we need to add more WebCallbacks interfaces
for bool type.

     }
 
     virtual void onError() override