[promisehook] Fire init hook for promise subclass

src/isolate.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_ISOLATE_H_
 #define V8_ISOLATE_H_
 
 #include <memory>
 #include <queue>
 
@@ skipping 1111 matching lines @@
   std::string GetTurboCfgFileName();
 
 #if TRACE_MAPS
   int GetNextUniqueSharedFunctionInfoId() { return next_unique_sfi_id_++; }
 #endif
 
   Address promise_hook_address() {
     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.
   Address is_tail_call_elimination_enabled_address() {
     return reinterpret_cast<Address>(&is_tail_call_elimination_enabled_);
   }
   bool is_tail_call_elimination_enabled() const {
     return is_tail_call_elimination_enabled_;
   }
   void SetTailCallEliminationEnabled(bool enabled);
@@ skipping 554 matching lines @@
 
   EmbeddedVector<char, 128> filename_;
   FILE* file_;
   int scope_depth_;
 };
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_ISOLATE_H_