[ES6 modules] Document ScriptModule

third_party/WebKit/Source/bindings/core/v8/ScriptModule.h

Index: third_party/WebKit/Source/bindings/core/v8/ScriptModule.h
diff --git a/third_party/WebKit/Source/bindings/core/v8/ScriptModule.h b/third_party/WebKit/Source/bindings/core/v8/ScriptModule.h
index fc51232d7f5d635423021db752bd6d01bb9ff217..6180243d642e29793ee4d5802665edae987a2115 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptModule.h
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptModule.h
@@ -14,6 +14,13 @@
 
 namespace blink {
 
+// ScriptModule wraps a handle to a v8::Module for use in core.
+//
+// Using ScriptModules needs a ScriptState and its scope to operate in. You
+// should always provide the same ScriptState and not try to reuse ScriptModules
+// across different contexts.
+// Currently all ScriptModule users can easily access its context Modulator, so
+// we use it to fill ScriptState in.

[haraken, 2017/04/04 05:28:26]

Would it be nicer to pass in the ScriptState to ScriptModule's constructor, save
it in ScriptModule's member and then use it for instantiate & evaluate?

[kouhei (in TOK), 2017/04/04 06:39:48]

I'd like to try ScriptModule lightweight w/o ScriptState for now, as then
Modulator will have to check ScriptState* is valid every time.

 class CORE_EXPORT ScriptModule final {
   DISALLOW_NEW_EXCEPT_PLACEMENT_NEW();