refactor(multiple): remove repeater strategy token

It seems like the `_VIEW_REPEATER_STRATEGY` token was introduced as a way to tree shake the different strategies depending on which components are used. While that's true, it also makes the APIs less ergonomic since users have to remember to import additional directives to use them.

These changes remove the token in favor of switching the strategy internally which means that we can't tree shake the strategies anymore, but they're relatively small (< 0.5kb minified, not gzipped).

Author: crisbeto

goldens/cdk/collections/index.api.md

@@ -5,7 +5,6 @@
 ```ts
 
 import * as i0 from '@angular/core';
-import { InjectionToken } from '@angular/core';
 import { IterableChangeRecord } from '@angular/core';
 import { IterableChanges } from '@angular/core';
 import { Observable } from 'rxjs';
@@ -110,9 +109,6 @@ export class UniqueSelectionDispatcher implements OnDestroy {
 // @public (undocumented)
 export type UniqueSelectionDispatcherListener = (id: string, name: string) => void;
 
-// @public
-export const _VIEW_REPEATER_STRATEGY: InjectionToken<_ViewRepeater<unknown, unknown, _ViewRepeaterItemContext<unknown>>>;
-
 // @public
 export interface _ViewRepeater<T, R, C extends _ViewRepeaterItemContext<T>> {
     // (undocumented)

goldens/cdk/table/index.api.md

@@ -262,7 +262,7 @@ export class CdkNoDataRow {
     static ɵfac: i0.ɵɵFactoryDeclaration<CdkNoDataRow, never>;
 }
 
-// @public
+// @public @deprecated
 export class CdkRecycleRows {
     // (undocumented)
     static ɵdir: i0.ɵɵDirectiveDeclaration<CdkRecycleRows, "cdk-table[recycleRows], table[cdk-table][recycleRows]", never, {}, {}, never, never, true, never>;
@@ -334,6 +334,8 @@ export class CdkTable<T> implements AfterContentInit, AfterContentChecked, Colle
     // (undocumented)
     static ngAcceptInputType_multiTemplateDataRows: unknown;
     // (undocumented)
+    static ngAcceptInputType_recycleRows: unknown;
+    // (undocumented)
     ngAfterContentChecked(): void;
     // (undocumented)
     ngAfterContentInit(): void;
@@ -345,6 +347,7 @@ export class CdkTable<T> implements AfterContentInit, AfterContentChecked, Colle
     // (undocumented)
     _noDataRowOutlet: NoDataRowOutlet;
     _outletAssigned(): void;
+    recycleRows: boolean;
     removeColumnDef(columnDef: CdkColumnDef): void;
     removeFooterRowDef(footerRowDef: CdkFooterRowDef): void;
     removeHeaderRowDef(headerRowDef: CdkHeaderRowDef): void;
@@ -366,9 +369,9 @@ export class CdkTable<T> implements AfterContentInit, AfterContentChecked, Colle
         end: number;
     }>;
     // (undocumented)
-    protected readonly _viewRepeater: _ViewRepeater<T, RenderRow<T>, RowContext<T>>;
+    protected _viewRepeater: _ViewRepeater<T, RenderRow<T>, RowContext<T>>;
     // (undocumented)
-    static ɵcmp: i0.ɵɵComponentDeclaration<CdkTable<any>, "cdk-table, table[cdk-table]", ["cdkTable"], { "trackBy": { "alias": "trackBy"; "required": false; }; "dataSource": { "alias": "dataSource"; "required": false; }; "multiTemplateDataRows": { "alias": "multiTemplateDataRows"; "required": false; }; "fixedLayout": { "alias": "fixedLayout"; "required": false; }; }, { "contentChanged": "contentChanged"; }, ["_noDataRow", "_contentColumnDefs", "_contentRowDefs", "_contentHeaderRowDefs", "_contentFooterRowDefs"], ["caption", "colgroup, col", "*"], true, never>;
+    static ɵcmp: i0.ɵɵComponentDeclaration<CdkTable<any>, "cdk-table, table[cdk-table]", ["cdkTable"], { "trackBy": { "alias": "trackBy"; "required": false; }; "dataSource": { "alias": "dataSource"; "required": false; }; "multiTemplateDataRows": { "alias": "multiTemplateDataRows"; "required": false; }; "fixedLayout": { "alias": "fixedLayout"; "required": false; }; "recycleRows": { "alias": "recycleRows"; "required": false; }; }, { "contentChanged": "contentChanged"; }, ["_noDataRow", "_contentColumnDefs", "_contentRowDefs", "_contentHeaderRowDefs", "_contentFooterRowDefs"], ["caption", "colgroup, col", "*"], true, never>;
     // (undocumented)
     static ɵfac: i0.ɵɵFactoryDeclaration<CdkTable<any>, never>;
 }

goldens/material/table/index.api.md

@@ -139,7 +139,7 @@ export class MatNoDataRow extends CdkNoDataRow {
     static ɵfac: i0.ɵɵFactoryDeclaration<MatNoDataRow, never>;
 }
 
-// @public
+// @public @deprecated
 export class MatRecycleRows {
     // (undocumented)
     static ɵdir: i0.ɵɵDirectiveDeclaration<MatRecycleRows, "mat-table[recycleRows], table[mat-table][recycleRows]", never, {}, {}, never, never, true, never>;

src/cdk/collections/view-repeater.ts

@@ -110,11 +110,3 @@ export interface _ViewRepeater<T, R, C extends _ViewRepeaterItemContext<T>> {
 
   detach(): void;
 }
-
-/**
- * Injection token for `_ViewRepeater`. This token is for use by Angular Material only.
- * @docs-private
- */
-export const _VIEW_REPEATER_STRATEGY = new InjectionToken<
-  _ViewRepeater<unknown, unknown, _ViewRepeaterItemContext<unknown>>
->('_ViewRepeater');

src/cdk/scrolling/virtual-for-of.ts

@@ -13,7 +13,6 @@ import {
   ListRange,
   isDataSource,
   _RecycleViewRepeaterStrategy,
-  _VIEW_REPEATER_STRATEGY,
   _ViewRepeaterItemInsertArgs,
 } from '../collections';
 import {
@@ -80,16 +79,14 @@ function getOffset(orientation: 'horizontal' | 'vertical', direction: 'start' |
  */
 @Directive({
   selector: '[cdkVirtualFor][cdkVirtualForOf]',
-  providers: [{provide: _VIEW_REPEATER_STRATEGY, useClass: _RecycleViewRepeaterStrategy}],
 })
 export class CdkVirtualForOf<T>
   implements CdkVirtualScrollRepeater<T>, CollectionViewer, DoCheck, OnDestroy
 {
   private _viewContainerRef = inject(ViewContainerRef);
   private _template = inject<TemplateRef<CdkVirtualForOfContext<T>>>(TemplateRef);
   private _differs = inject(IterableDiffers);
-  private _viewRepeater =
-    inject<_RecycleViewRepeaterStrategy<T, T, CdkVirtualForOfContext<T>>>(_VIEW_REPEATER_STRATEGY);
+  private _viewRepeater = new _RecycleViewRepeaterStrategy<T, T, CdkVirtualForOfContext<T>>();
   private _viewport = inject(CdkVirtualScrollViewport, {skipSelf: true});
 
   /** Emits when the rendered view of the data changes. */

src/cdk/table/table.ts

@@ -13,7 +13,6 @@ import {
   _DisposeViewRepeaterStrategy,
   _RecycleViewRepeaterStrategy,
   isDataSource,
-  _VIEW_REPEATER_STRATEGY,
   _ViewRepeater,
   _ViewRepeaterItemChange,
   _ViewRepeaterItemInsertArgs,
@@ -87,11 +86,11 @@ import {CDK_TABLE} from './tokens';
 /**
  * Enables the recycle view repeater strategy, which reduces rendering latency. Not compatible with
  * tables that animate rows.
+ *
+ * @deprecated This directive is a no-op and will be removed.
+ * @breaking-change 23.0.0
  */
-@Directive({
-  selector: 'cdk-table[recycleRows], table[cdk-table][recycleRows]',
-  providers: [{provide: _VIEW_REPEATER_STRATEGY, useClass: _RecycleViewRepeaterStrategy}],
-})
+@Directive({selector: 'cdk-table[recycleRows], table[cdk-table][recycleRows]'})
 export class CdkRecycleRows {}
 
 /** Interface used to provide an outlet for rows to be inserted into. */
@@ -268,7 +267,6 @@ export interface RenderRow<T> {
   changeDetection: ChangeDetectionStrategy.Default,
   providers: [
     {provide: CDK_TABLE, useExisting: CdkTable},
-    {provide: _VIEW_REPEATER_STRATEGY, useClass: _DisposeViewRepeaterStrategy},
     // Prevent nested tables from seeing this table's StickyPositioningListener.
     {provide: STICKY_POSITIONING_LISTENER, useValue: null},
   ],
@@ -282,8 +280,7 @@ export class CdkTable<T>
   protected readonly _elementRef = inject(ElementRef);
   protected readonly _dir = inject(Directionality, {optional: true});
   private _platform = inject(Platform);
-  protected readonly _viewRepeater =
-    inject<_ViewRepeater<T, RenderRow<T>, RowContext<T>>>(_VIEW_REPEATER_STRATEGY);
+  protected _viewRepeater: _ViewRepeater<T, RenderRow<T>, RowContext<T>>;
   private readonly _viewportRuler = inject(ViewportRuler);
   protected readonly _stickyPositioningListener = inject<StickyPositioningListener>(
     STICKY_POSITIONING_LISTENER,
@@ -544,6 +541,12 @@ export class CdkTable<T>
   }
   private _fixedLayout: boolean = false;
 
+  /**
+   * Whether rows should be recycled which reduces latency, but is not compatible with tables
+   * that animate rows. Note that this input cannot change after the table is initialized.
+   */
+  @Input({transform: booleanAttribute}) recycleRows = false;
+
   /**
    * Emits when the table completes rendering a set of data rows based on the latest data from the
    * data source, even if the set of rows is empty.
@@ -628,6 +631,9 @@ export class CdkTable<T>
   }
 
   ngAfterContentInit() {
+    this._viewRepeater = this.recycleRows
+      ? new _RecycleViewRepeaterStrategy()
+      : new _DisposeViewRepeaterStrategy();
     this._hasInitialized = true;
   }
 

src/material/table/table.ts

@@ -16,20 +16,16 @@ import {
   NoDataRowOutlet,
   FooterRowOutlet,
 } from '@angular/cdk/table';
-import {
-  _DisposeViewRepeaterStrategy,
-  _RecycleViewRepeaterStrategy,
-  _VIEW_REPEATER_STRATEGY,
-} from '@angular/cdk/collections';
+import {_DisposeViewRepeaterStrategy, _RecycleViewRepeaterStrategy} from '@angular/cdk/collections';
 
 /**
  * Enables the recycle view repeater strategy, which reduces rendering latency. Not compatible with
  * tables that animate rows.
+ *
+ * @deprecated This directive is a no-op and will be removed.
+ * @breaking-change 23.0.0
  */
-@Directive({
-  selector: 'mat-table[recycleRows], table[mat-table][recycleRows]',
-  providers: [{provide: _VIEW_REPEATER_STRATEGY, useClass: _RecycleViewRepeaterStrategy}],
-})
+@Directive({selector: 'mat-table[recycleRows], table[mat-table][recycleRows]'})
 export class MatRecycleRows {}
 
 @Component({
@@ -75,9 +71,6 @@ export class MatRecycleRows {}
   providers: [
     {provide: CdkTable, useExisting: MatTable},
     {provide: CDK_TABLE, useExisting: MatTable},
-    // TODO(michaeljamesparsons) Abstract the view repeater strategy to a directive API so this code
-    //  is only included in the build if used.
-    {provide: _VIEW_REPEATER_STRATEGY, useClass: _DisposeViewRepeaterStrategy},
     // Prevent nested tables from seeing this table's StickyPositioningListener.
     {provide: STICKY_POSITIONING_LISTENER, useValue: null},
   ],