fix(multiple): use eager change detection

Changes all the usages of `ChangeDetectionStrategy.Default` to use `Eager` instead.

Author: crisbeto

src/cdk/dialog/dialog-container.ts

@@ -57,7 +57,7 @@ export function throwDialogContentAlreadyAttachedError() {
   encapsulation: ViewEncapsulation.None,
   // Using OnPush for dialogs caused some G3 sync issues. Disabled until we can track them down.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   imports: [CdkPortalOutlet],
   host: {
     'class': 'cdk-dialog-container',

src/cdk/table/row.ts

@@ -321,7 +321,7 @@ export class CdkCellOutlet implements OnDestroy {
   },
   // See note on CdkTable for explanation on why this uses the default change detection strategy.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   imports: [CdkCellOutlet],
 })
@@ -337,7 +337,7 @@ export class CdkHeaderRow {}
   },
   // See note on CdkTable for explanation on why this uses the default change detection strategy.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   imports: [CdkCellOutlet],
 })
@@ -353,7 +353,7 @@ export class CdkFooterRow {}
   },
   // See note on CdkTable for explanation on why this uses the default change detection strategy.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   imports: [CdkCellOutlet],
 })

src/cdk/table/table.ts

@@ -275,7 +275,7 @@ export interface RenderRow<T> {
   // The view for `MatTable` consists entirely of templates declared in other views. As they are
   // declared elsewhere, they are checked when their declaration points are checked.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   providers: [
     {provide: CDK_TABLE, useExisting: CdkTable},
     // Prevent nested tables from seeing this table's StickyPositioningListener.

src/cdk/table/text-column.ts

@@ -52,7 +52,7 @@ import {TEXT_COLUMN_OPTIONS, TextColumnOptions} from './tokens';
   // mean's the template in the table's view will not have the updated value (and in fact will cause
   // an ExpressionChangedAfterItHasBeenCheckedError).
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   imports: [CdkColumnDef, CdkHeaderCellDef, CdkHeaderCell, CdkCellDef, CdkCell],
 })
 export class CdkTextColumn<T> implements OnDestroy, OnInit {

src/cdk/tree/tree.ts

@@ -112,7 +112,7 @@ type RenderingData<T> =
   // The view for `CdkTree` consists entirely of templates declared in other views. As they are
   // declared elsewhere, they are checked when their declaration points are checked.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   imports: [CdkTreeNodeOutlet],
 })
 export class CdkTree<T, K = T>

src/material/bottom-sheet/bottom-sheet-container.ts

@@ -35,7 +35,7 @@ const EXIT_ANIMATION = '_mat-bottom-sheet-exit';
   // they were stamped out. This means that we can't have the bottom sheet container be OnPush,
   // because it might cause the sheets that were opened from a template not to be out of date.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   host: {
     'class': 'mat-bottom-sheet-container',

src/material/dialog/dialog-container.ts

@@ -48,7 +48,7 @@ export const CLOSE_ANIMATION_DURATION = 75;
   encapsulation: ViewEncapsulation.None,
   // Disabled for consistency with the non-MDC dialog container.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   imports: [CdkPortalOutlet],
   host: {
     'class': 'mat-mdc-dialog-container mdc-dialog',

src/material/snack-bar/snack-bar-container.ts

@@ -51,7 +51,7 @@ const EXIT_ANIMATION = '_mat-snack-bar-exit';
   // where they were stamped out. This means that we can't have the snack bar container be OnPush,
   // because it might cause snack bars that were opened from a template not to be out of date.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   imports: [CdkPortalOutlet],
   host: {

src/material/table/row.ts

@@ -80,7 +80,7 @@ export class MatRowDef<T> extends CdkRowDef<T> {}
   },
   // See note on CdkTable for explanation on why this uses the default change detection strategy.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   exportAs: 'matHeaderRow',
   providers: [{provide: CdkHeaderRow, useExisting: MatHeaderRow}],
@@ -98,7 +98,7 @@ export class MatHeaderRow extends CdkHeaderRow {}
   },
   // See note on CdkTable for explanation on why this uses the default change detection strategy.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   exportAs: 'matFooterRow',
   providers: [{provide: CdkFooterRow, useExisting: MatFooterRow}],
@@ -116,7 +116,7 @@ export class MatFooterRow extends CdkFooterRow {}
   },
   // See note on CdkTable for explanation on why this uses the default change detection strategy.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   encapsulation: ViewEncapsulation.None,
   exportAs: 'matRow',
   providers: [{provide: CdkRow, useExisting: MatRow}],

src/material/table/table.ts

@@ -77,7 +77,7 @@ export class MatRecycleRows {}
   encapsulation: ViewEncapsulation.None,
   // See note on CdkTable for explanation on why this uses the default change detection strategy.
   // tslint:disable-next-line:validate-decorators
-  changeDetection: ChangeDetectionStrategy.Default,
+  changeDetection: ChangeDetectionStrategy.Eager,
   imports: [HeaderRowOutlet, DataRowOutlet, NoDataRowOutlet, FooterRowOutlet],
 })
 export class MatTable<T> extends CdkTable<T> {