fix(cdk/scrolling): avoid injector errors

Fixes some "Injector is already destroyed" errors.

Author: crisbeto

src/cdk/scrolling/virtual-scroll-viewport.ts

@@ -14,6 +14,7 @@ import {
   ChangeDetectionStrategy,
   ChangeDetectorRef,
   Component,
+  DestroyRef,
   ElementRef,
   inject,
   Inject,
@@ -201,6 +202,17 @@ export class CdkVirtualScrollViewport extends CdkVirtualScrollable implements On
       this.elementRef.nativeElement.classList.add('cdk-virtual-scrollable');
       this.scrollable = this;
     }
+
+    this._injector.get(DestroyRef).onDestroy(() => {
+      this._isDestroyed = true;
+      this.detach();
+      this._scrollStrategy.detach();
+
+      // Complete all subjects
+      this._renderedRangeSubject.complete();
+      this._detachedSubject.complete();
+      this._viewportChanges.unsubscribe();
+    });
   }
 
   override ngOnInit() {
@@ -242,20 +254,6 @@ export class CdkVirtualScrollViewport extends CdkVirtualScrollable implements On
     );
   }
 
-  override ngOnDestroy() {
-    this.detach();
-    this._scrollStrategy.detach();
-
-    // Complete all subjects
-    this._renderedRangeSubject.complete();
-    this._detachedSubject.complete();
-    this._viewportChanges.unsubscribe();
-
-    this._isDestroyed = true;
-
-    super.ngOnDestroy();
-  }
-
   /** Attaches a `CdkVirtualScrollRepeater` to this viewport. */
   attach(forOf: CdkVirtualScrollRepeater<any>) {
     if (this._forOf && (typeof ngDevMode === 'undefined' || ngDevMode)) {

tools/public_api_guard/cdk/scrolling.md

@@ -197,8 +197,6 @@ export class CdkVirtualScrollViewport extends CdkVirtualScrollable implements On
     // (undocumented)
     static ngAcceptInputType_appendOnly: unknown;
     // (undocumented)
-    ngOnDestroy(): void;
-    // (undocumented)
     ngOnInit(): void;
     get orientation(): "horizontal" | "vertical";
     set orientation(orientation: 'horizontal' | 'vertical');