From 6c44222fa1658b6912b69fc1916944bda0d18faf Mon Sep 17 00:00:00 2001
From: Kristiyan Kostadinov <crisbeto@abv.bg>
Date: Fri, 3 Jun 2022 10:37:47 +0200
Subject: [PATCH] refactor(core): don't use patched data in
 LifecycleHooksFeature (#46237)

We don't need to read patched data off the component instance since we already know what the `LView` is.

PR Close #46237
---
 packages/core/src/render3/component.ts | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/packages/core/src/render3/component.ts b/packages/core/src/render3/component.ts
index e1ff73a1b3b..f632ef985bc 100644
--- a/packages/core/src/render3/component.ts
+++ b/packages/core/src/render3/component.ts
@@ -14,7 +14,6 @@ import {Sanitizer} from '../sanitization/sanitizer';
 import {assertDefined, assertIndexInRange} from '../util/assert';
 
 import {assertComponentType} from './assert';
-import {readPatchedLView} from './context_discovery';
 import {getComponentDef} from './definition';
 import {diPublicInInjector, getOrCreateNodeInjectorForNode} from './di';
 import {throwProviderNotFoundError} from './errors_di';
@@ -27,7 +26,7 @@ import {domRendererFactory3, Renderer3, RendererFactory3} from './interfaces/ren
 import {RElement} from './interfaces/renderer_dom';
 import {CONTEXT, HEADER_OFFSET, LView, LViewFlags, RootContext, RootContextFlags, TVIEW, TViewType} from './interfaces/view';
 import {writeDirectClass, writeDirectStyle} from './node_manipulation';
-import {enterView, getCurrentTNode, leaveView, setSelectedIndex} from './state';
+import {enterView, getCurrentTNode, getLView, leaveView, setSelectedIndex} from './state';
 import {computeStaticStyling} from './styling/static_styling';
 import {setUpAttributes} from './util/attrs_utils';
 import {publishDefaultGlobalUtils} from './util/global_utils';
@@ -228,7 +227,11 @@ export function createRootComponent<T>(
   rootContext.components.push(component);
   componentView[CONTEXT] = component;
 
-  hostFeatures && hostFeatures.forEach((feature) => feature(component, componentDef));
+  if (hostFeatures !== null) {
+    for (const feature of hostFeatures) {
+      feature(component, componentDef);
+    }
+  }
 
   // We want to generate an empty QueryList for root content queries for backwards
   // compatibility with ViewEngine.
@@ -279,13 +282,10 @@ export function createRootContext(
  * renderComponent(AppComponent, {hostFeatures: [LifecycleHooksFeature]});
  * ```
  */
-export function LifecycleHooksFeature(component: any, def: ComponentDef<any>): void {
-  const lView = readPatchedLView(component)!;
-  ngDevMode && assertDefined(lView, 'LView is required');
-  const tView = lView[TVIEW];
+export function LifecycleHooksFeature(): void {
   const tNode = getCurrentTNode()!;
   ngDevMode && assertDefined(tNode, 'TNode is required');
-  registerPostOrderHooks(tView, tNode);
+  registerPostOrderHooks(getLView()[TVIEW], tNode);
 }
 
 /**