refactor(compiler-cli): move DelegatingPerfRecorder initialization into constructor (#54834)

frost-cy · alxhub · commit 42318e7c3ddc · 2024-03-15T15:10:46.000-07:00
Move the initialization of class field `DelegatingPerfRecorder` into the constructor. This fixes the error : `TypeError: Cannot read properties of undefined (reading 'eventCount')` This is blocking the roll-out of public class. PR Close #54834
diff --git a/packages/compiler-cli/src/ngtsc/core/src/compiler.ts b/packages/compiler-cli/src/ngtsc/core/src/compiler.ts
@@ -279,7 +279,7 @@ export class NgCompiler {
    * `NgCompiler` use a `DelegatingPerfRecorder` so the `PerfRecorder` they write to can be updated
    * with each fresh compilation.
    */
-  private delegatingPerfRecorder = new DelegatingPerfRecorder(this.perfRecorder);
+  private delegatingPerfRecorder: DelegatingPerfRecorder;
 
   /**
    * Convert a `CompilationTicket` into an `NgCompiler` instance for the requested compilation.
@@ -334,6 +334,7 @@ export class NgCompiler {
       readonly usePoisonedData: boolean,
       private livePerfRecorder: ActivePerfRecorder,
   ) {
+    this.delegatingPerfRecorder = new DelegatingPerfRecorder(this.perfRecorder);
     this.enableTemplateTypeChecker =
         enableTemplateTypeChecker || (options['_enableTemplateTypeChecker'] ?? false);
     // TODO(crisbeto): remove this flag and base `enableBlockSyntax` on the `angularCoreVersion`.
