fix: resolve errors with closure

Also don't patch all "on" properties if a list of properties is being
passed in. Otherwise various global variables in window get clobbered.

Closes #722

Author: vikerman

Diff for: lib/browser/property-descriptor.ts

@@ -25,8 +25,8 @@ export function propertyDescriptorPatch(_global: any) {
     if (isBrowser) {
       patchOnProperties(window, eventNames);
       patchOnProperties(Document.prototype, eventNames);
-      if (typeof SVGElement !== 'undefined') {
-        patchOnProperties(SVGElement.prototype, eventNames);
+      if (typeof (<any>window)['SVGElement'] !== 'undefined') {
+        patchOnProperties((<any>window)['SVGElement'].prototype, eventNames);
       }
       patchOnProperties(HTMLElement.prototype, eventNames);
     }

Diff for: lib/common/utils.ts

@@ -154,19 +154,20 @@ export function patchProperty(obj: any, prop: string) {
 }
 
 export function patchOnProperties(obj: any, properties: string[]) {
-  const onProperties = [];
-  for (const prop in obj) {
-    if (prop.substr(0, 2) == 'on') {
-      onProperties.push(prop);
-    }
-  }
-  for (let j = 0; j < onProperties.length; j++) {
-    patchProperty(obj, onProperties[j]);
-  }
   if (properties) {
     for (let i = 0; i < properties.length; i++) {
       patchProperty(obj, 'on' + properties[i]);
     }
+  } else {
+    const onProperties = [];
+    for (const prop in obj) {
+      if (prop.substr(0, 2) == 'on') {
+        onProperties.push(prop);
+      }
+    }
+    for (let j = 0; j < onProperties.length; j++) {
+      patchProperty(obj, onProperties[j]);
+    }
   }
 }