refactor(util): remove getter functions for Polymer/ShadyCSS/customEl…

…ements

BREAKING CHANGE:
The utility functions to get the Polymer/ShadyCSS/customElements properties have been removed. Instead typings are provided in the repo to access window.Polymer and window.ShadyCSS

origami/src/origami.ts

@@ -2,11 +2,10 @@ export * from './forms/iron-control.directive';
 export * from './polymer.module';
 export * from './style/shared-styles-host';
 export * from './templates/polymer-template.directive';
-export * from './util/customElements';
+export * from './types/Polymer';
+export * from './types/ShadyCSS';
 export * from './util/descriptors';
 export * from './util/getCustomElementClass';
 export * from './util/getTagName';
-export * from './util/Polymer';
-export * from './util/ShadyCSS';
 export * from './util/unwrapPolymerEvent';
 export * from './util/webcomponents';

origami/src/style/shared-styles-host.ts

@@ -1,7 +1,7 @@
 import { Inject } from '@angular/core';
 import { DOCUMENT, ɵDomSharedStylesHost as DomSharedStylesHost } from '@angular/platform-browser';
 
-import { getShadyCSS } from '../util/ShadyCSS';
+import '../types/ShadyCSS';
 
 /**
  * poylmer-webpack-loader will insert <style> and <custom-style> elements into the body, while
@@ -73,7 +73,7 @@ export class PolymerDomSharedStylesHost extends DomSharedStylesHost {
 
   private addStylesToShadyCSS() {
     // Importing Polymer will import ShadyCSS, but it is not required
-    const shadyCss = getShadyCSS();
+    const shadyCss = window.ShadyCSS;
     if (shadyCss) {
       this.hostNodes.forEach(hostNode => {
         Array.from(hostNode.childNodes).forEach((childNode: Element) => {

origami/src/templates/polymer-template.directive.spec.ts

@@ -5,7 +5,7 @@ import { ComponentFixture, TestBed, async } from '@angular/core/testing';
 import 'iron-list/iron-list.html';
 import 'paper-checkbox/paper-checkbox.html';
 
-import { getPolymer } from '../util/Polymer';
+import '../types/Polymer';
 import { unwrapPolymerEvent } from '../util/unwrapPolymerEvent';
 import { PolymerTemplateDirective } from './polymer-template.directive';
 
@@ -77,7 +77,7 @@ describe('PolymerTemplateDirective', () => {
       bindFixture.detectChanges();
       bindFixture.whenStable().then(() => {
         const ironList = bindFixture.debugElement.nativeElement.querySelector('iron-list');
-        getPolymer().RenderStatus.afterNextRender({}, () => {
+        window.Polymer.RenderStatus.afterNextRender({}, () => {
           const checkboxes = Array.from(ironList.querySelectorAll('paper-checkbox'));
           expect(checkboxes.length).toEqual(3);
           bindFixture.componentInstance.ngChecked = true;

origami/src/templates/polymer-template.directive.ts

@@ -1,8 +1,8 @@
 // tslint:disable:no-string-literal
 import { Directive, ElementRef, Input, NgZone, OnInit } from '@angular/core';
 
+import '../types/Polymer';
 import { wrapAndDefineDescriptor } from '../util/descriptors';
-import { getPolymer } from '../util/Polymer';
 import { unwrapPolymerEvent } from '../util/unwrapPolymerEvent';
 import { webcomponentsReady } from '../util/webcomponents';
 
@@ -79,7 +79,7 @@ export class PolymerTemplateDirective implements OnInit {
     if (templateInfo && templateInfo.hostProps) {
       Object.keys(templateInfo.hostProps).forEach(hostProp => {
         // Polymer -> Angular
-        const eventName = `_host_${getPolymer().CaseMap.camelToDashCase(hostProp)}-changed`;
+        const eventName = `_host_${window.Polymer.CaseMap.camelToDashCase(hostProp)}-changed`;
         this.template.addEventListener(eventName, (e: CustomEvent) => {
           this.zone.run(() => {
             const value = unwrapPolymerEvent(e);

origami/src/util/Polymer.ts → origami/src/types/Polymer.ts

@@ -66,7 +66,3 @@ declare global {
     Polymer: Polymer;
   }
 }
-
-export function getPolymer(): Polymer {
-  return window.Polymer;
-}

origami/src/util/ShadyCSS.ts → origami/src/types/ShadyCSS.ts

@@ -13,7 +13,3 @@ declare global {
     ShadyCSS: ShadyCSS;
   }
 }
-
-export function getShadyCSS(): ShadyCSS {
-  return window.ShadyCSS;
-}

origami/src/util/getCustomElementClass.spec.ts

@@ -1,12 +1,11 @@
 import 'polymer/polymer.html';
 
-import { getCustomElements } from './customElements';
+import '../types/Polymer';
 import { getCustomElementClass } from './getCustomElementClass';
-import { getPolymer } from './Polymer';
 
 describe('getCustomElementClass()', () => {
   it('should return constructor function for the element', () => {
-    const CustomElement = getPolymer()({ // tslint:disable-line:variable-name
+    const CustomElement = window.Polymer()({ // tslint:disable-line:variable-name
       is: 'custom-element'
     });
 

origami/src/util/getCustomElementClass.ts

@@ -1,12 +1,11 @@
 import { ElementRef } from '@angular/core';
 
-import { getCustomElements } from './customElements';
 import { getTagName } from './getTagName';
 
 // tslint:disable-next-line:ban-types
 export function getCustomElementClass(elementRef?: ElementRef): Function|void {
   if (elementRef && elementRef.nativeElement) {
-    const klass = getCustomElements().get(elementRef.nativeElement.tagName.toLowerCase());
+    const klass = window.customElements.get(elementRef.nativeElement.tagName.toLowerCase());
     if (klass) {
       return klass;
     } else {