Specs: custom element constructor argument shouldn't clash with the module-global 'module' identifi…

sky/specs/modules.md

 Sky Module System
 =================
 
 This document describes the Sky module system.
 
 Overview
 --------
 
 The Sky module system is based on the ``import`` element. In its
 most basic form, you import a module as follows:
@@ skipping 38 matching lines @@
 
 ### IDL ###
 
 ```javascript
 dictionary InternalElementOptions {
   String tagName;
   Boolean shadow = false;
   Object prototype = Element;
 }
 interface InternalElementConstructorWithoutShadow {
-  constructor (Module module);
+  constructor (Module hostModule);
   attribute String tagName;
 }
 interface InternalElementConstructorWithShadow {
-  constructor (Module module);
+  constructor (Module hostModule);
   attribute String tagName;
   attribute Boolean shadow;
 }
 typedef ElementRegistrationOptions (InternalElementOptions or
                                     InternalElementConstructorWithoutShadow or
                                     InternalElementConstructorWithShadow);
 
 abstract class AbstractModule : EventTarget {
-  readonly attribute Document document; // O(1) // the Documentof the module or application
+  readonly attribute Document document; // O(1) // the Document of the module or application
   Promise<any> import(String url); // O(Yikes) // returns the module's exports
   private Array<Module> getImports(); O(N) // returns the Module objects of all the imported modules
 
   readonly attribute String url;
 
   ElementConstructor registerElement(Object options); // O(1)
   // if you call registerElement() with an object that was created by
   // registerElement(), it just returns the object after registering it,
   // rather than creating a new constructor
   // otherwise, it proceeds as follows:
   //  - if options is a Function (i.e. it is either an
   //    InternalElementConstructorWithoutShadow object or an
   //    InternalElementConstructorWithShadow object), then let
   //    constructor be that function, and let prototype be that
   //    functions's prototype; otherwise, let constructor be a no-op
   //    function and let prototype be the prototype property of the
   //    object passed in (the InternalElementOptions; prototype
   //    defaults to Element).
   //  - let shadow be option's shadow property's value coerced to a
   //    boolean, if the property is present, or else the value false.
-  //  - let shadow be option's tagName property's value.
-  //  - create a new Function that:
-  //      - throws if not called as a constructor
-  //      - creates an actual element object (the C++-backed object)
-  //      - initialises the shadow tree if shadow on the options is
-  //        true
-  //      - calls constructor, if it's not null, with the module as
-  //        the argument
-  //      - is marked as created by registerElement() so that it can
-  //        be recognised if used as an argument to registerElement()
+  //  - let tagName be option's tagName property's value.
+  //  - create a new Function that acts as if it had the signature of
+  //    the constructors in the ElementConstructor interface, and that
+  //    runs the follows steps when called:
+  //      - throw if not called as a constructor
+  //      - create an actual element object (the C++-backed object)
+  //        called tagName, along with the specified attributes
+  //      - initialise the shadow tree if shadow is true
+  //      - call constructor, if it's not null, with the module
+  //        within which the new element is being constructed as the
+  //        argument
+  //      - append all the specified children
+  //  - mark that new Function as created by registerElement() so that
+  //    it can be recognised if used as an argument to
+  //    registerElement()
   //  - let that new Function's prototype be the aforementioned prototype
   //  - let that new Function have tagName and shadow properties set to
   //    the aforementioned tagName and shadow
   //  - register the new tagName with this constructor
   //  - return the new Function (which is, not coincidentally, an
   //    InternalElementConstructorWithShadow)
 
   readonly attribute ScriptElement? currentScript; // O(1) // returns the <script> element currently being executed if any, and if it's in this module; else null
 }
 
@@ skipping 32 matching lines @@
 Where ``name_1`` through ``name_n`` are the names bound to the
 various named imports in the script element's document,
 ``source_code`` is the text content of the script element,
 ``source_module`` is the ``Module`` object of the script element's
 module, and ``value_1`` through ``value_n`` are the values
 exported by the various named imports in the script element's
 document.
 
 When an import fails to load, the ``as`` name for the import gets
 bound to ``undefined``.