[promisehook] Fire init hook for promise subclass

src/isolate.h

Index: src/isolate.h
diff --git a/src/isolate.h b/src/isolate.h
index 36c22bbddd18f4342ed42dd3b4d11fccd4dab9f0..652a60b0b8450fa7a148571895a71e2ab190ba33 100644
--- a/src/isolate.h
+++ b/src/isolate.h
@@ -1129,7 +1129,7 @@ class Isolate {
     return reinterpret_cast<Address>(&promise_hook_);
   }
   void SetPromiseHook(PromiseHook hook);
-  void RunPromiseHook(PromiseHookType type, Handle<JSPromise> promise,
+  void RunPromiseHook(PromiseHookType type, Handle<JSObject> promise,

[Dan Ehrenberg, 2016/12/20 22:45:18]

I don't see how the promise here would not be a JSPromise. Doesn't this patch
only change subclass instantiation? Given that Promise is a base class, it's
hard for me to see how it would end up not getting called on a real JSPromise.

[gsathya, 2016/12/20 23:47:26]

You're right. I was under the wrong impression that subclasses aren't the same
type. Thanks!

                       Handle<Object> parent);
 
   // Support for dynamically disabling tail call elimination.