[ESNext] Implement DynamicImportCall

include/v8.h

Index: include/v8.h
diff --git a/include/v8.h b/include/v8.h
index d7b7205ef0a0b18e341343875f614ca18fed6610..c956bc0ca120aef2a9bd7570d5871aedb2ee7a35 100644
--- a/include/v8.h
+++ b/include/v8.h
@@ -1071,6 +1071,14 @@ class V8_EXPORT Module {
    * Returns the completion value.
    */
   V8_WARN_UNUSED_RESULT MaybeLocal<Value> Evaluate(Local<Context> context);
+
+  static bool FinishDynamicImportSuccess(Local<Context> context,

[adamk, 2017/03/15 21:36:43]

It looks like this returns false if PromiseResolve somehow throws? It should
have V8_WARN_UNUSED_RESULT in that case. Same goes below.

The idea is that the caller will have to do something about the exception.

[gsathya, 2017/03/16 00:59:24]

Done.

+                                         Local<Promise> promise,
+                                         Local<Module> module);
+
+  static bool FinishDynamicImportFailure(Local<Context> context,
+                                         Local<Promise> promise,
+                                         Local<Value> exception);
 };
 
 /**
@@ -5787,6 +5795,11 @@ typedef void (*BeforeCallEnteredCallback)(Isolate*);
 typedef void (*CallCompletedCallback)(Isolate*);
 typedef void (*DeprecatedCallCompletedCallback)();
 
+typedef void (*HostImportModuleDynamicallyCallback)(Isolate* isolate,
+                                                    Local<String> referrer,
+                                                    Local<String> specifier,
+                                                    Local<Promise> promise);

[adamk, 2017/03/15 21:36:43]

As previously discussed, can you type the Promise differently, so that the
embedder doesn't accidentally resolve it in the wrong way?

+
 /**
  * PromiseHook with type kInit is called when a new promise is
  * created. When a new promise is created as part of the chain in the
@@ -6336,7 +6349,8 @@ class V8_EXPORT Isolate {
           add_histogram_sample_callback(nullptr),
           array_buffer_allocator(nullptr),
           external_references(nullptr),
-          allow_atomics_wait(true) {}
+          allow_atomics_wait(true),
+          host_import_module_dynamically_callback_(nullptr) {}
 
     /**
      * The optional entry_hook allows the host application to provide the
@@ -6399,6 +6413,12 @@ class V8_EXPORT Isolate {
      * this isolate.
      */
     bool allow_atomics_wait;
+
+    /**
+     *  TODO(gsathya): Write doc
+     */
+    HostImportModuleDynamicallyCallback
+        host_import_module_dynamically_callback_;
   };