@angular/core 9.1.3 → 9.1.4

package/bundles/core.umd.js

@@ -1,5 +1,5 @@
 /**
- * @license Angular v9.1.3
+ * @license Angular v9.1.4
  * (c) 2010-2020 Google LLC. https://angular.io/
  * License: MIT
  */
@@ -2678,7 +2678,7 @@
     function setBindingRootForHostBindings(bindingRootIndex, currentDirectiveIndex) {
         var lFrame = instructionState.lFrame;
         lFrame.bindingIndex = lFrame.bindingRootIndex = bindingRootIndex;
-        lFrame.currentDirectiveIndex = currentDirectiveIndex;
+        setCurrentDirectiveIndex(currentDirectiveIndex);
     }
     /**
      * When host binding is executing this points to the directive index.
@@ -2688,6 +2688,24 @@
     function getCurrentDirectiveIndex() {
         return instructionState.lFrame.currentDirectiveIndex;
     }
+    /**
+     * Sets an index of a directive whose `hostBindings` are being processed.
+     *
+     * @param currentDirectiveIndex `TData` index where current directive instance can be found.
+     */
+    function setCurrentDirectiveIndex(currentDirectiveIndex) {
+        instructionState.lFrame.currentDirectiveIndex = currentDirectiveIndex;
+    }
+    /**
+     * Retrieve the current `DirectiveDef` which is active when `hostBindings` instruction is being
+     * executed.
+     *
+     * @param tData Current `TData` where the `DirectiveDef` will be looked up at.
+     */
+    function getCurrentDirectiveDef(tData) {
+        var currentDirectiveIndex = instructionState.lFrame.currentDirectiveIndex;
+        return currentDirectiveIndex === -1 ? null : tData[currentDirectiveIndex];
+    }
     function getCurrentQueryIndex() {
         return instructionState.lFrame.currentQueryIndex;
     }
@@ -6660,7 +6678,9 @@
         pipeRegistry, //
         firstChild, //
         schemas, //
-        consts) {
+        consts, //
+        incompleteFirstPass //
+        ) {
             this.type = type;
             this.id = id;
             this.blueprint = blueprint;
@@ -6691,6 +6711,7 @@
             this.firstChild = firstChild;
             this.schemas = schemas;
             this.consts = consts;
+            this.incompleteFirstPass = incompleteFirstPass;
         }
         Object.defineProperty(TView.prototype, "template_", {
             get: function () {
@@ -7702,6 +7723,14 @@
                 renderChildComponents(lView, components);
             }
         }
+        catch (error) {
+            // If we didn't manage to get past the first template pass due to
+            // an error, mark the view as corrupted so we can try to recover.
+            if (tView.firstCreatePass) {
+                tView.incompleteFirstPass = true;
+            }
+            throw error;
+        }
         finally {
             lView[FLAGS] &= ~4 /* CreationMode */;
             leaveView();
@@ -7907,8 +7936,13 @@
      * @returns TView
      */
     function getOrCreateTComponentView(def) {
-        return def.tView ||
-            (def.tView = createTView(1 /* Component */, -1, def.template, def.decls, def.vars, def.directiveDefs, def.pipeDefs, def.viewQuery, def.schemas, def.consts));
+        var tView = def.tView;
+        // Create a TView if there isn't one, or recreate it if the first create pass didn't
+        // complete successfuly since we can't know for sure whether it's in a usable shape.
+        if (tView === null || tView.incompleteFirstPass) {
+            return def.tView = createTView(1 /* Component */, -1, def.template, def.decls, def.vars, def.directiveDefs, def.pipeDefs, def.viewQuery, def.schemas, def.consts);
+        }
+        return tView;
     }
     /**
      * Creates a TView instance
@@ -7961,7 +7995,9 @@
             typeof pipes === 'function' ? pipes() : pipes, // pipeRegistry: PipeDefList|null,
             null, // firstChild: TNode|null,
             schemas, // schemas: SchemaMetadata[]|null,
-            consts) : // consts: TConstants|null
+            consts, // consts: TConstants|null
+            false // incompleteFirstPass: boolean
+            ) :
             {
                 type: type,
                 id: viewIndex,
@@ -7993,6 +8029,7 @@
                 firstChild: null,
                 schemas: schemas,
                 consts: consts,
+                incompleteFirstPass: false
             };
     }
     function createViewBlueprint(bindingStartIndex, initialViewLength) {
@@ -8514,11 +8551,13 @@
         var expando = tView.expandoInstructions;
         var firstCreatePass = tView.firstCreatePass;
         var elementIndex = tNode.index - HEADER_OFFSET;
+        var currentDirectiveIndex = getCurrentDirectiveIndex();
         try {
             setSelectedIndex(elementIndex);
-            for (var i = start; i < end; i++) {
-                var def = tView.data[i];
-                var directive = lView[i];
+            for (var dirIndex = start; dirIndex < end; dirIndex++) {
+                var def = tView.data[dirIndex];
+                var directive = lView[dirIndex];
+                setCurrentDirectiveIndex(dirIndex);
                 if (def.hostBindings !== null || def.hostVars !== 0 || def.hostAttrs !== null) {
                     invokeHostBindingsInCreationMode(def, directive);
                 }
@@ -8529,6 +8568,7 @@
         }
         finally {
             setSelectedIndex(-1);
+            setCurrentDirectiveIndex(currentDirectiveIndex);
         }
     }
     /**
@@ -9120,9 +9160,17 @@
      * There are cases where the sub component's renderer needs to be included
      * instead of the current renderer (see the componentSyntheticHost* instructions).
      */
-    function loadComponentRenderer(tNode, lView) {
-        var componentLView = unwrapLView(lView[tNode.index]);
-        return componentLView[RENDERER];
+    function loadComponentRenderer(currentDef, tNode, lView) {
+        // TODO(FW-2043): the `currentDef` is null when host bindings are invoked while creating root
+        // component (see packages/core/src/render3/component.ts). This is not consistent with the process
+        // of creating inner components, when current directive index is available in the state. In order
+        // to avoid relying on current def being `null` (thus special-casing root component creation), the
+        // process of creating root component should be unified with the process of creating inner
+        // components.
+        if (currentDef === null || isComponentDef(currentDef)) {
+            lView = unwrapLView(lView[tNode.index]);
+        }
+        return lView[RENDERER];
     }
     /** Handles an error thrown in an LView. */
     function handleError(lView, error) {
@@ -15040,7 +15088,7 @@
         return ɵɵlistener;
     }
     /**
-     * Registers a synthetic host listener (e.g. `(@foo.start)`) on a component.
+     * Registers a synthetic host listener (e.g. `(@foo.start)`) on a component or directive.
      *
      * This instruction is for compatibility purposes and is designed to ensure that a
      * synthetic host listener (e.g. `@HostListener('@foo.start')`) properly gets rendered
@@ -15064,8 +15112,9 @@
         if (useCapture === void 0) { useCapture = false; }
         var tNode = getPreviousOrParentTNode();
         var lView = getLView();
-        var renderer = loadComponentRenderer(tNode, lView);
         var tView = getTView();
+        var currentDef = getCurrentDirectiveDef(tView.data);
+        var renderer = loadComponentRenderer(currentDef, tNode, lView);
         listenerInternal(tView, lView, renderer, tNode, eventName, listenerFn, useCapture, eventTargetResolver);
         return ɵɵcomponentHostSyntheticListener;
     }
@@ -16841,7 +16890,7 @@
      * @param isClassBased `true` if `class` (`false` if `style`)
      */
     function wrapInStaticStylingKey(tData, tNode, stylingKey, isClassBased) {
-        var hostDirectiveDef = getHostDirectiveDef(tData);
+        var hostDirectiveDef = getCurrentDirectiveDef(tData);
         var residual = isClassBased ? tNode.residualClasses : tNode.residualStyles;
         if (hostDirectiveDef === null) {
             // We are in template node.
@@ -17070,16 +17119,6 @@
         }
         return stylingKey === undefined ? null : stylingKey;
     }
-    /**
-     * Retrieve the current `DirectiveDef` which is active when `hostBindings` style instruction is
-     * being executed (or `null` if we are in `template`.)
-     *
-     * @param tData Current `TData` where the `DirectiveDef` will be looked up at.
-     */
-    function getHostDirectiveDef(tData) {
-        var currentDirectiveIndex = getCurrentDirectiveIndex();
-        return currentDirectiveIndex === -1 ? null : tData[currentDirectiveIndex];
-    }
     /**
      * Convert user input to `KeyValueArray`.
      *
@@ -18717,7 +18756,7 @@
         return ɵɵhostProperty;
     }
     /**
-     * Updates a synthetic host binding (e.g. `[@foo]`) on a component.
+     * Updates a synthetic host binding (e.g. `[@foo]`) on a component or directive.
      *
      * This instruction is for compatibility purposes and is designed to ensure that a
      * synthetic host binding (e.g. `@HostBinding('@foo')`) properly gets rendered in
@@ -18743,7 +18782,8 @@
         if (bindingUpdated(lView, bindingIndex, value)) {
             var tView = getTView();
             var tNode = getSelectedTNode();
-            var renderer = loadComponentRenderer(tNode, lView);
+            var currentDef = getCurrentDirectiveDef(tView.data);
+            var renderer = loadComponentRenderer(currentDef, tNode, lView);
             elementPropertyInternal(tView, tNode, lView, propName, value, renderer, sanitizer, true);
             ngDevMode && storePropertyBindingMetadata(tView.data, tNode, propName, bindingIndex);
         }
@@ -20319,7 +20359,7 @@
     /**
      * @publicApi
      */
-    var VERSION = new Version('9.1.3');
+    var VERSION = new Version('9.1.4');
 
     /**
      * @license