fix: make material work with noUnusedParameters

* Recently Angular core fixed their `noUnusedParameters` warnings in angular/angular#15532
* When using Angular with Material and the users has `noUnusedParameters` enabled the library will be checked and will throw.

Fixes angular#4443

Author: devversion

src/demo-app/dialog/dialog-demo.ts

@@ -38,7 +38,7 @@ export class DialogDemo {
     // Possible useful example for the open and closeAll events.
     // Adding a class to the body if a dialog opens and
     // removing it after all open dialogs are closed
-    dialog.afterOpen.subscribe((ref: MdDialogRef<any>) => {
+    dialog.afterOpen.subscribe(() => {
       if (!doc.body.classList.contains('no-scroll')) {
         doc.body.classList.add('no-scroll');
       }

src/demo-app/style/style-demo.html

@@ -1,10 +1,10 @@
 <button #b class="demo-focusable" cdkMonitorElementFocus>focus me!</button>
 <button (click)="b.focus()">focus programmatically</button>
 
-<button (click)="fom.focusVia(b, renderer, 'mouse')">focusVia: mouse</button>
-<button (click)="fom.focusVia(b, renderer, 'touch')">focusVia: touch</button>
-<button (click)="fom.focusVia(b, renderer, 'keyboard')">focusVia: keyboard</button>
-<button (click)="fom.focusVia(b, renderer, 'program')">focusVia: program</button>
+<button (click)="fom.focusVia(b, 'mouse')">focusVia: mouse</button>
+<button (click)="fom.focusVia(b, 'touch')">focusVia: touch</button>
+<button (click)="fom.focusVia(b, 'keyboard')">focusVia: keyboard</button>
+<button (click)="fom.focusVia(b, 'program')">focusVia: program</button>
 
 <div>Active classes: {{b.classList}}</div>
 

src/demo-app/style/style-demo.ts

@@ -9,5 +9,5 @@ import {FocusOriginMonitor} from '@angular/material';
   styleUrls: ['style-demo.css'],
 })
 export class StyleDemo {
-  constructor(public renderer: Renderer2, public fom: FocusOriginMonitor) {}
+  constructor(public fom: FocusOriginMonitor) {}
 }

src/lib/autocomplete/autocomplete-trigger.ts

@@ -83,7 +83,7 @@ export class MdAutocompleteTrigger implements ControlValueAccessor, OnDestroy {
   private _manuallyFloatingPlaceholder = false;
 
   /** View -> model callback called when value changes */
-  _onChange = (value: any) => {};
+  _onChange: (value: any) => void = () => {};
 
   /** View -> model callback called when autocomplete has been touched */
   _onTouched = () => {};

src/lib/autocomplete/autocomplete.spec.ts

@@ -67,7 +67,7 @@ describe('MdAutocomplete', () => {
         }},
         {provide: Dir, useFactory: () => ({value: dir})},
         {provide: ScrollDispatcher, useFactory: () => {
-          return {scrolled: (delay: number, callback: () => any) => {
+          return {scrolled: (_delay: number, callback: () => any) => {
             return scrolledSubject.asObservable().subscribe(callback);
           }};
         }}

src/lib/button-toggle/button-toggle.ts

@@ -80,7 +80,7 @@ export class MdButtonToggleGroup implements AfterViewInit, ControlValueAccessor
    * The method to be called in order to update ngModel.
    * Now `ngModel` binding is not supported in multiple selection mode.
    */
-  private _controlValueAccessorChangeFn: (value: any) => void = (value) => {};
+  private _controlValueAccessorChangeFn: (value: any) => void = () => {};
 
   /** onTouch function registered via registerOnTouch (ControlValueAccessor). */
   onTouched: () => any = () => {};

src/lib/checkbox/checkbox.spec.ts

@@ -814,8 +814,8 @@ class SingleCheckbox {
   checkboxColor: string = 'primary';
   checkboxValue: string = 'single_checkbox';
 
-  onCheckboxClick(event: Event) {}
-  onCheckboxChange(event: MdCheckboxChange) {}
+  onCheckboxClick(_event: Event) {}
+  onCheckboxChange(_event: MdCheckboxChange) {}
 }
 
 /** Simple component for testing an MdCheckbox with ngModel. */

src/lib/checkbox/checkbox.ts

@@ -185,7 +185,7 @@ export class MdCheckbox extends _MdCheckboxMixinBase
 
   private _color: string;
 
-  private _controlValueAccessorChangeFn: (value: any) => void = (value) => {};
+  private _controlValueAccessorChangeFn: (value: any) => void = () => {};
 
   /** Reference to the focused state ripple. */
   private _focusRipple: RippleRef;
@@ -393,7 +393,7 @@ export class MdCheckbox extends _MdCheckboxMixinBase
 
   /** Focuses the checkbox. */
   focus(): void {
-    this._focusOriginMonitor.focusVia(this._inputElement.nativeElement, this._renderer, 'keyboard');
+    this._focusOriginMonitor.focusVia(this._inputElement.nativeElement, 'keyboard');
   }
 
   _onInteractionEvent(event: Event) {

src/lib/chips/chip-list.spec.ts

@@ -236,11 +236,8 @@ describe('MdChipList', () => {
 class StaticChipList {
   name: string = 'Test';
   selectable: boolean = true;
-  remove: Number;
+  remove: number;
 
-  chipSelect(index: Number) {
-  }
-
-  chipDeselect(index: Number) {
-  }
+  chipSelect(_index: number) {}
+  chipDeselect(_index: number) {}
 }

src/lib/chips/chip.spec.ts

@@ -135,17 +135,10 @@ class SingleChip {
   selected: boolean = false;
   shouldShow: boolean = true;
 
-  chipFocus(event: MdChipEvent) {
-  }
-
-  chipDestroy(event: MdChipEvent) {
-  }
-
-  chipSelect(event: MdChipEvent) {
-  }
-
-  chipDeselect(event: MdChipEvent) {
-  }
+  chipFocus(_event: MdChipEvent) {}
+  chipDestroy(_event: MdChipEvent) {}
+  chipSelect(_event: MdChipEvent) {}
+  chipDeselect(_event: MdChipEvent) {}
 }
 
 @Component({

src/lib/core/data-table/data-table.spec.ts

@@ -2,7 +2,6 @@ import {async, ComponentFixture, TestBed} from '@angular/core/testing';
 import {Component, ViewChild} from '@angular/core';
 import {CdkTable} from './data-table';
 import {CollectionViewer, DataSource} from './data-source';
-import {CommonModule} from '@angular/common';
 import {Observable} from 'rxjs/Observable';
 import {BehaviorSubject} from 'rxjs/BehaviorSubject';
 import {customMatchers} from '../testing/jasmine-matchers';
@@ -213,7 +212,7 @@ function getHeaderCells(tableElement: Element): Element[] {
 }
 
 const tableCustomMatchers: jasmine.CustomMatcherFactories = {
-  toMatchTableContent: function(util, customEqualityTesters) {
+  toMatchTableContent: () => {
     return {
       compare: function (tableElement: Element, expectedTableContent: any[]) {
         const missedExpectations = [];

src/lib/core/datetime/native-date-adapter.ts

@@ -31,7 +31,7 @@ const DEFAULT_DAY_OF_WEEK_NAMES = {
 
 /** Creates an array and fills it with values. */
 function range<T>(length: number, valueFunction: (index: number) => T): T[] {
-  return Array.apply(null, Array(length)).map((v: undefined, i: number) => valueFunction(i));
+  return Array.apply(null, Array(length)).map((_v: undefined, i: number) => valueFunction(i));
 }
 
 
@@ -123,7 +123,7 @@ export class NativeDateAdapter extends DateAdapter<Date> {
     return new Date();
   }
 
-  parse(value: any, parseFormat: Object): Date | null {
+  parse(value: any): Date | null {
     // We have no way using the native JS Date to set the parse format or locale, so we ignore these
     // parameters.
     let timestamp = typeof value == 'number' ? value : Date.parse(value);

src/lib/core/overlay/scroll/close-scroll-strategy.spec.ts

@@ -8,7 +8,6 @@ import {
   OverlayState,
   OverlayRef,
   OverlayModule,
-  ScrollStrategy,
   ScrollDispatcher,
   CloseScrollStrategy,
 } from '../../core';
@@ -24,7 +23,7 @@ describe('CloseScrollStrategy', () => {
       imports: [OverlayModule, PortalModule, OverlayTestModule],
       providers: [
         {provide: ScrollDispatcher, useFactory: () => {
-          return {scrolled: (delay: number, callback: () => any) => {
+          return {scrolled: (_delay: number, callback: () => any) => {
             return scrolledSubject.asObservable().subscribe(callback);
           }};
         }}

src/lib/core/overlay/scroll/reposition-scroll-strategy.spec.ts

@@ -8,7 +8,6 @@ import {
   OverlayState,
   OverlayRef,
   OverlayModule,
-  ScrollStrategy,
   ScrollDispatcher,
   RepositionScrollStrategy,
 } from '../../core';
@@ -24,7 +23,7 @@ describe('RepositionScrollStrategy', () => {
       imports: [OverlayModule, PortalModule, OverlayTestModule],
       providers: [
         {provide: ScrollDispatcher, useFactory: () => {
-          return {scrolled: (delay: number, callback: () => any) => {
+          return {scrolled: (_delay: number, callback: () => any) => {
             return scrolledSubject.asObservable().subscribe(callback);
           }};
         }}

src/lib/core/overlay/scroll/scroll-dispatcher.ts

@@ -105,7 +105,7 @@ export class ScrollDispatcher {
   getScrollContainers(elementRef: ElementRef): Scrollable[] {
     const scrollingContainers: Scrollable[] = [];
 
-    this.scrollableReferences.forEach((subscription: Subscription, scrollable: Scrollable) => {
+    this.scrollableReferences.forEach((_subscription: Subscription, scrollable: Scrollable) => {
       if (this.scrollableContainsElement(scrollable, elementRef)) {
         scrollingContainers.push(scrollable);
       }

src/lib/core/style/focus-origin-monitor.spec.ts

@@ -131,7 +131,7 @@ describe('FocusOriginMonitor', () => {
   }));
 
   it('focusVia keyboard should simulate keyboard focus', async(() => {
-    focusOriginMonitor.focusVia(buttonElement, buttonRenderer, 'keyboard');
+    focusOriginMonitor.focusVia(buttonElement, 'keyboard');
     fixture.detectChanges();
 
     setTimeout(() => {
@@ -148,7 +148,7 @@ describe('FocusOriginMonitor', () => {
   }));
 
   it('focusVia mouse should simulate mouse focus', async(() => {
-    focusOriginMonitor.focusVia(buttonElement, buttonRenderer, 'mouse');
+    focusOriginMonitor.focusVia(buttonElement, 'mouse');
     fixture.detectChanges();
 
     setTimeout(() => {
@@ -165,7 +165,7 @@ describe('FocusOriginMonitor', () => {
   }));
 
   it('focusVia mouse should simulate mouse focus', async(() => {
-    focusOriginMonitor.focusVia(buttonElement, buttonRenderer, 'touch');
+    focusOriginMonitor.focusVia(buttonElement, 'touch');
     fixture.detectChanges();
 
     setTimeout(() => {
@@ -182,7 +182,7 @@ describe('FocusOriginMonitor', () => {
   }));
 
   it('focusVia program should simulate programmatic focus', async(() => {
-    focusOriginMonitor.focusVia(buttonElement, buttonRenderer, 'program');
+    focusOriginMonitor.focusVia(buttonElement, 'program');
     fixture.detectChanges();
 
     setTimeout(() => {
@@ -464,7 +464,7 @@ class PlainButton {
   template: `<button cdkMonitorElementFocus (cdkFocusChange)="focusChanged($event)"></button>`
 })
 class ButtonWithFocusClasses {
-  focusChanged(origin: FocusOrigin) {}
+  focusChanged(_origin: FocusOrigin) {}
 }
 
 

src/lib/core/style/focus-origin-monitor.ts

@@ -129,7 +129,7 @@ export class FocusOriginMonitor {
    * @param renderer The renderer to use to invoke the focus method on the element.
    * @param origin The focus origin.
    */
-  focusVia(element: HTMLElement, renderer: Renderer2, origin: FocusOrigin): void {
+  focusVia(element: HTMLElement, origin: FocusOrigin): void {
     this._setOriginForCurrentEventQueue(origin);
     element.focus();
   }

src/lib/core/testing/jasmine-matchers.ts

@@ -2,8 +2,7 @@
  * Collection of useful custom jasmine matchers for tests.
  */
 export const customMatchers: jasmine.CustomMatcherFactories = {
-  toBeRole: function(util: jasmine.MatchersUtil,
-                     customEqualityTesters: jasmine.CustomEqualityTester[]) {
+  toBeRole: () => {
     return {
       compare: function (element: Element, expectedRole: string) {
         const result: jasmine.CustomMatcherResult = {pass: false};

src/lib/core/testing/type-in-element.ts

@@ -6,7 +6,7 @@ import {dispatchFakeEvent} from './dispatch-events';
  * @param value Value to be set on the input.
  * @param element Element onto which to set the value.
  */
-export function typeInElement(value: string, element: HTMLInputElement, autoFocus = true) {
+export function typeInElement(value: string, element: HTMLInputElement) {
   element.focus();
   element.value = value;
   dispatchFakeEvent(element, 'input');

src/lib/datepicker/datepicker-input.ts

@@ -121,7 +121,7 @@ export class MdDatepickerInput<D> implements AfterContentInit, ControlValueAcces
 
   _onTouched = () => {};
 
-  private _cvaOnChange = (value: any) => {};
+  private _cvaOnChange: (value: any) => void = () => {};
 
   private _validatorOnChange = () => {};
 

src/lib/grid-list/tile-styler.ts

@@ -6,7 +6,7 @@ import {TileCoordinator} from './tile-coordinator';
  * Tile Coordinator.
  * @docs-private
  */
-export class TileStyler {
+export abstract class TileStyler {
   _gutterSize: string;
   _rows: number = 0;
   _rowspan: number = 0;
@@ -122,11 +122,12 @@ export class TileStyler {
    * This method will be implemented by each type of TileStyler.
    * @docs-private
    */
-  setRowStyles(tile: MdGridTile, rowIndex: number, percentWidth: number, gutterWidth: number) {}
+  abstract setRowStyles(tile: MdGridTile, rowIndex: number, percentWidth: number,
+                        gutterWidth: number);
 
   /**
    * Calculates the computed height and returns the correct style property to set.
-   * This method will be implemented by each type of TileStyler.
+   * This method can be implemented by each type of TileStyler.
    * @docs-private
    */
   getComputedHeight(): [string, string] { return null; }
@@ -147,8 +148,7 @@ export class FixedTileStyler extends TileStyler {
     this.fixedRowHeight = normalizeUnits(this.fixedRowHeight);
   }
 
-  setRowStyles(tile: MdGridTile, rowIndex: number, percentWidth: number,
-               gutterWidth: number): void {
+  setRowStyles(tile: MdGridTile, rowIndex: number): void {
     tile._setStyle('top', this.getTilePosition(this.fixedRowHeight, rowIndex));
     tile._setStyle('height', calc(this.getTileSize(this.fixedRowHeight, tile.rowspan)));
   }
@@ -215,8 +215,7 @@ export class RatioTileStyler extends TileStyler {
  */
 export class FitTileStyler extends TileStyler {
 
-  setRowStyles(tile: MdGridTile, rowIndex: number, percentWidth: number,
-               gutterWidth: number): void {
+  setRowStyles(tile: MdGridTile, rowIndex: number): void {
     // Percent of the available vertical space that one row takes up.
     let percentHeightPerTile = 100 / this._rowspan;
 
@@ -229,6 +228,7 @@ export class FitTileStyler extends TileStyler {
     tile._setStyle('top', this.getTilePosition(baseTileHeight, rowIndex));
     tile._setStyle('height', calc(this.getTileSize(baseTileHeight, tile.rowspan)));
   }
+
 }
 
 

src/lib/icon/icon-registry.ts

@@ -245,7 +245,7 @@ export class MdIconRegistry {
         .filter(iconSetConfig => !iconSetConfig.svgElement)
         .map(iconSetConfig =>
             this._loadSvgIconSetFromConfig(iconSetConfig)
-                .catch((err: any, caught: Observable<SVGElement>): Observable<SVGElement> => {
+                .catch((err: any): Observable<SVGElement> => {
                   let url =
                       this._sanitizer.sanitize(SecurityContext.RESOURCE_URL, iconSetConfig.url);
 
@@ -263,7 +263,7 @@ export class MdIconRegistry {
     // Fetch all the icon set URLs. When the requests complete, every IconSet should have a
     // cached SVG element (unless the request failed), and we can check again for the icon.
     return Observable.forkJoin(iconSetFetchRequests)
-        .map((ignoredResults: any) => {
+        .map(() => {
           const foundIcon = this._extractIconWithNameFromAnySet(name, iconSetConfigs);
           if (!foundIcon) {
             throw getMdIconNameNotFoundError(name);

src/lib/radio/radio.spec.ts

@@ -143,7 +143,7 @@ describe('MdRadio', () => {
       expect(radioInstances[0].checked).toBe(false);
 
       let spies = radioInstances
-        .map((value, index) => jasmine.createSpy(`onChangeSpy ${index}`));
+        .map((_value, index) => jasmine.createSpy(`onChangeSpy ${index}`));
 
       spies.forEach((spy, index) => radioInstances[index].change.subscribe(spy));
 

src/lib/radio/radio.ts

@@ -95,7 +95,7 @@ export class MdRadioGroup extends _MdRadioGroupMixinBase
   private _disabled: boolean = false;
 
   /** The method to be called in order to update ngModel */
-  _controlValueAccessorChangeFn: (value: any) => void = (value) => {};
+  _controlValueAccessorChangeFn: (value: any) => void = () => {};
 
   /**
    * onTouch function registered via registerOnTouch (ControlValueAccessor).
@@ -462,7 +462,7 @@ export class MdRadioButton implements OnInit, AfterViewInit, OnDestroy {
 
   /** Focuses the radio button. */
   focus(): void {
-    this._focusOriginMonitor.focusVia(this._inputElement.nativeElement, this._renderer, 'keyboard');
+    this._focusOriginMonitor.focusVia(this._inputElement.nativeElement, 'keyboard');
   }
 
   /**