fixup! feat(material-experimental): MDC-based version of dialog

Potential fix for test failure in g3

Author: devversion

src/material-experimental/mdc-dialog/dialog-container.ts

@@ -9,7 +9,6 @@
 import {FocusMonitor, FocusTrapFactory} from '@angular/cdk/a11y';
 import {DOCUMENT} from '@angular/common';
 import {
-  AfterViewInit,
   ChangeDetectionStrategy,
   ChangeDetectorRef,
   Component,
@@ -46,9 +45,7 @@ import {cssClasses, numbers} from '@material/dialog';
     '[class._mat-animation-noopable]': '!_animationsEnabled',
   },
 })
-export class MatDialogContainer extends _MatDialogContainerBase implements
-    AfterViewInit, OnDestroy {
-
+export class MatDialogContainer extends _MatDialogContainerBase implements OnDestroy {
   /** Whether animations are enabled. */
   _animationsEnabled: boolean = this._animationMode !== 'NoopAnimations';
 
@@ -75,7 +72,10 @@ export class MatDialogContainer extends _MatDialogContainerBase implements
     super(elementRef, focusTrapFactory, changeDetectorRef, document, config, focusMonitor);
   }
 
-  ngAfterViewInit() {
+  _initializeWithAttachedContent() {
+    // Delegate to the original dialog-container initialization (i.e. saving the
+    // previous element, setting up the focus trap and moving focus to the container).
+    super._initializeWithAttachedContent();
     // Note: Usually we would be able to use the MDC dialog foundation here to handle
     // the dialog animation for us, but there are a few reasons why we just leverage
     // their styles and not use the runtime foundation code:
@@ -106,7 +106,11 @@ export class MatDialogContainer extends _MatDialogContainerBase implements
       this._waitForAnimationToComplete(this._openAnimationDuration, this._finishDialogOpen);
     } else {
       this._hostElement.classList.add(cssClasses.OPEN);
-      this._finishDialogOpen();
+      // Note: We could immediately finish the dialog opening here with noop animations,
+      // but we defer until next tick so that consumers can subscribe to `afterOpened`.
+      // Executing this immediately would mean that `afterOpened` emits synchronously
+      // on `dialog.open` before the consumer had a change to subscribe to `afterOpened`.
+      Promise.resolve().then(() => this._finishDialogOpen());
     }
   }