fix(EventTarget): apply the patch even if Window is not defined

vicb · vicb · commit 32c6df979929 · 2015-10-06T10:25:59.000-07:00
diff --git a/lib/patch/event-target.js b/lib/patch/event-target.js
@@ -10,7 +10,7 @@ function apply() {
   // Note: EventTarget is not available in all browsers,
   // if it's not available, we instead patch the APIs in the IDL that inherit from EventTarget
   } else {
-    var apis = [ 
+    var apis = [
       'ApplicationCache',
       'EventSource',
       'FileReader',
@@ -25,24 +25,29 @@ function apply() {
       'TextTrackCue',
       'TextTrackList',
       'WebKitNamedFlow',
-      'Window',
       'Worker',
       'WorkerGlobalScope',
       'XMLHttpRequest',
       'XMLHttpRequestEventTarget',
       'XMLHttpRequestUpload'
     ];
 
-    apis.forEach(function(thing) {
-      var obj = global[thing] && global[thing].prototype;
+    apis.forEach(function(api) {
+      var proto = global[api] && global[api].prototype;
 
       // Some browsers e.g. Android 4.3's don't actually implement
       // the EventTarget methods for all of these e.g. FileReader.
-      // In this case, there is nothing to patch. 
-      if (obj && obj.addEventListener) {
-        utils.patchEventTargetMethods(obj);
+      // In this case, there is nothing to patch.
+      if (proto && proto.addEventListener) {
+        utils.patchEventTargetMethods(proto);
       }
     });
+
+    // Patch the methods on `window` instead of `Window.prototype`
+    // `Window` is not accessible on Android 4.3
+    if (typeof(window) !== 'undefined') {
+      utils.patchEventTargetMethods(window);
+    }
   }
 }
 
