refactor: avoid casting document to any in constructor (#10775)

Removes all the places where we were casting the `Document` to `any` when passing it in to the constructor, in order to avoid AoT compilation issues. It seems like this is no longer necessary as of 6.0.

Author: crisbeto

src/cdk-experimental/dialog/dialog-container.ts

@@ -109,7 +109,7 @@ export class CdkDialogContainer extends BasePortalOutlet {
     private _elementRef: ElementRef,
     private _focusTrapFactory: FocusTrapFactory,
     private _changeDetectorRef: ChangeDetectorRef,
-    @Optional() @Inject(DOCUMENT) private _document: any) {
+    @Optional() @Inject(DOCUMENT) private _document: Document) {
     super();
   }
 

src/cdk/a11y/aria-describer/aria-describer.ts

@@ -55,11 +55,7 @@ let messagesContainer: HTMLElement | null = null;
  */
 @Injectable({providedIn: 'root'})
 export class AriaDescriber {
-  private _document: Document;
-
-  constructor(@Inject(DOCUMENT) _document: any) {
-    this._document = _document;
-  }
+  constructor(@Inject(DOCUMENT) private _document: Document) {}
 
   /**
    * Adds to the host element an aria-describedby reference to a hidden element that contains
@@ -213,7 +209,8 @@ export class AriaDescriber {
 
 
 /** @docs-private @deprecated @deletion-target 7.0.0 */
-export function ARIA_DESCRIBER_PROVIDER_FACTORY(parentDispatcher: AriaDescriber, _document: any) {
+export function ARIA_DESCRIBER_PROVIDER_FACTORY(parentDispatcher: AriaDescriber,
+  _document: Document) {
   return parentDispatcher || new AriaDescriber(_document);
 }
 
@@ -223,7 +220,7 @@ export const ARIA_DESCRIBER_PROVIDER = {
   provide: AriaDescriber,
   deps: [
     [new Optional(), new SkipSelf(), AriaDescriber],
-    DOCUMENT as InjectionToken<any>
+    DOCUMENT as InjectionToken<Document>
   ],
   useFactory: ARIA_DESCRIBER_PROVIDER_FACTORY
 };

src/cdk/a11y/focus-trap/focus-trap.ts

@@ -285,15 +285,10 @@ export class FocusTrap {
 /** Factory that allows easy instantiation of focus traps. */
 @Injectable({providedIn: 'root'})
 export class FocusTrapFactory {
-  private _document: Document;
-
   constructor(
       private _checker: InteractivityChecker,
       private _ngZone: NgZone,
-      @Inject(DOCUMENT) _document: any) {
-
-    this._document = _document;
-  }
+      @Inject(DOCUMENT) private _document: Document) {}
 
   /**
    * Creates a focus-trapped region around the given element.
@@ -314,8 +309,6 @@ export class FocusTrapFactory {
   exportAs: 'cdkTrapFocus',
 })
 export class CdkTrapFocus implements OnDestroy, AfterContentInit {
-  private _document: Document;
-
   /** Underlying FocusTrap instance. */
   focusTrap: FocusTrap;
 
@@ -339,7 +332,7 @@ export class CdkTrapFocus implements OnDestroy, AfterContentInit {
   constructor(
       private _elementRef: ElementRef,
       private _focusTrapFactory: FocusTrapFactory,
-      @Inject(DOCUMENT) _document: any) {
+      @Inject(DOCUMENT) private _document: Document) {
 
     this._document = _document;
     this.focusTrap = this._focusTrapFactory.create(this._elementRef.nativeElement, true);

src/cdk/a11y/live-announcer/live-announcer.ts

@@ -27,7 +27,7 @@ export class LiveAnnouncer implements OnDestroy {
 
   constructor(
       @Optional() @Inject(LIVE_ANNOUNCER_ELEMENT_TOKEN) elementToken: any,
-      @Inject(DOCUMENT) private _document: any) {
+      @Inject(DOCUMENT) private _document: Document) {
 
     // We inject the live element as `any` because the constructor signature cannot reference
     // browser globals (HTMLElement) on non-browser environments, since having a class decorator
@@ -83,7 +83,7 @@ export class LiveAnnouncer implements OnDestroy {
 
 /** @docs-private @deprecated @deletion-target 7.0.0 */
 export function LIVE_ANNOUNCER_PROVIDER_FACTORY(
-    parentDispatcher: LiveAnnouncer, liveElement: any, _document: any) {
+    parentDispatcher: LiveAnnouncer, liveElement: any, _document: Document) {
   return parentDispatcher || new LiveAnnouncer(liveElement, _document);
 }
 

src/cdk/overlay/keyboard/overlay-keyboard-dispatcher.ts

@@ -29,12 +29,9 @@ export class OverlayKeyboardDispatcher implements OnDestroy {
   /** Currently attached overlays in the order they were attached. */
   _attachedOverlays: OverlayRef[] = [];
 
-  private _document: Document;
   private _isAttached: boolean;
 
-  constructor(@Inject(DOCUMENT) document: any) {
-    this._document = document;
-  }
+  constructor(@Inject(DOCUMENT) private _document: Document) {}
 
   ngOnDestroy() {
     this._detach();
@@ -97,7 +94,7 @@ export class OverlayKeyboardDispatcher implements OnDestroy {
 
 /** @docs-private @deprecated @deletion-target 7.0.0 */
 export function OVERLAY_KEYBOARD_DISPATCHER_PROVIDER_FACTORY(
-    dispatcher: OverlayKeyboardDispatcher, _document: any) {
+    dispatcher: OverlayKeyboardDispatcher, _document: Document) {
   return dispatcher || new OverlayKeyboardDispatcher(_document);
 }
 
@@ -111,7 +108,7 @@ export const OVERLAY_KEYBOARD_DISPATCHER_PROVIDER = {
 
     // Coerce to `InjectionToken` so that the `deps` match the "shape"
     // of the type expected by Angular
-    DOCUMENT as InjectionToken<any>
+    DOCUMENT as InjectionToken<Document>
   ],
   useFactory: OVERLAY_KEYBOARD_DISPATCHER_PROVIDER_FACTORY
 };

src/cdk/overlay/overlay-container.ts

@@ -22,7 +22,7 @@ import {
 export class OverlayContainer implements OnDestroy {
   protected _containerElement: HTMLElement;
 
-  constructor(@Inject(DOCUMENT) private _document: any) {}
+  constructor(@Inject(DOCUMENT) private _document: Document) {}
 
   ngOnDestroy() {
     if (this._containerElement && this._containerElement.parentNode) {
@@ -57,7 +57,7 @@ export class OverlayContainer implements OnDestroy {
 
 /** @docs-private @deprecated @deletion-target 7.0.0 */
 export function OVERLAY_CONTAINER_PROVIDER_FACTORY(parentContainer: OverlayContainer,
-  _document: any) {
+  _document: Document) {
   return parentContainer || new OverlayContainer(_document);
 }
 
@@ -67,7 +67,8 @@ export const OVERLAY_CONTAINER_PROVIDER = {
   provide: OverlayContainer,
   deps: [
     [new Optional(), new SkipSelf(), OverlayContainer],
-    DOCUMENT as InjectionToken<any> // We need to use the InjectionToken somewhere to keep TS happy
+    // We need to use the InjectionToken somewhere to keep TS happy
+    DOCUMENT as InjectionToken<Document>
   ],
   useFactory: OVERLAY_CONTAINER_PROVIDER_FACTORY
 };

src/cdk/overlay/overlay.ts

@@ -51,7 +51,7 @@ export class Overlay {
               private _appRef: ApplicationRef,
               private _injector: Injector,
               private _ngZone: NgZone,
-              @Inject(DOCUMENT) private _document: any,
+              @Inject(DOCUMENT) private _document: Document,
               private _directionality: Directionality) { }
 
   /**

src/cdk/overlay/position/overlay-position-builder.ts

@@ -20,7 +20,7 @@ import {GlobalPositionStrategy} from './global-position-strategy';
 export class OverlayPositionBuilder {
   constructor(
     private _viewportRuler: ViewportRuler,
-    @Inject(DOCUMENT) private _document: any) { }
+    @Inject(DOCUMENT) private _document: Document) { }
 
   /**
    * Creates a global position strategy.

src/cdk/overlay/scroll/block-scroll-strategy.ts

@@ -17,11 +17,8 @@ export class BlockScrollStrategy implements ScrollStrategy {
   private _previousHTMLStyles = { top: '', left: '' };
   private _previousScrollPosition: { top: number, left: number };
   private _isEnabled = false;
-  private _document: Document;
 
-  constructor(private _viewportRuler: ViewportRuler, document: any) {
-    this._document = document;
-  }
+  constructor(private _viewportRuler: ViewportRuler, private _document: Document) {}
 
   /** Attaches this scroll strategy to an overlay. */
   attach() { }

src/cdk/overlay/scroll/scroll-strategy-options.ts

@@ -26,15 +26,11 @@ import {
  */
 @Injectable({providedIn: 'root'})
 export class ScrollStrategyOptions {
-  private _document: Document;
-
   constructor(
     private _scrollDispatcher: ScrollDispatcher,
     private _viewportRuler: ViewportRuler,
     private _ngZone: NgZone,
-    @Inject(DOCUMENT) document: any) {
-      this._document = document;
-    }
+    @Inject(DOCUMENT) private _document: Document) {}
 
   /** Do nothing on scroll. */
   noop = () => new NoopScrollStrategy();