fix(WebWorker): Fix patching in WebWorker

jteplitz · vicb · commit 2cc59d8b1b87 · 2015-08-07T18:24:20.000-07:00
diff --git a/lib/patch/property-descriptor.js b/lib/patch/property-descriptor.js
@@ -6,6 +6,11 @@ var utils = require('../utils');
 var eventNames = 'copy cut paste abort blur focus canplay canplaythrough change click contextmenu dblclick drag dragend dragenter dragleave dragover dragstart drop durationchange emptied ended input invalid keydown keypress keyup load loadeddata loadedmetadata loadstart message mousedown mouseenter mouseleave mousemove mouseout mouseover mouseup pause play playing progress ratechange reset scroll seeked seeking select show stalled submit suspend timeupdate volumechange waiting mozfullscreenchange mozfullscreenerror mozpointerlockchange mozpointerlockerror error webglcontextrestored webglcontextlost webglcontextcreationerror'.split(' ');
 
 function apply() {
+  if (utils.isWebWorker()){
+    // on WebWorker so don't apply patch
+    return;
+  }
+
   if (canPatchViaPropertyDescriptor()) {
     // for browsers that we can patch the descriptor:  Chrome & Firefox
     var onEventNames = eventNames.map(function (property) {
diff --git a/lib/patch/register-element.js b/lib/patch/register-element.js
@@ -1,9 +1,10 @@
 'use strict';
 
 var _redefineProperty = require('./define-property')._redefineProperty;
+var utils = require("../utils");
 
 function apply() {
-  if (!('registerElement' in global.document)) {
+  if (utils.isWebWorker() || !('registerElement' in global.document)) {
     return;
   }
 
diff --git a/lib/utils.js b/lib/utils.js
@@ -29,6 +29,10 @@ function patchPrototype(obj, fnNames) {
   });
 };
 
+function isWebWorker() {
+  return (typeof document === "undefined");
+}
+
 function patchProperty(obj, prop) {
   var desc = Object.getOwnPropertyDescriptor(obj, prop) || {
     enumerable: true,
@@ -178,5 +182,6 @@ module.exports = {
   patchProperty: patchProperty,
   patchProperties: patchProperties,
   patchEventTargetMethods: patchEventTargetMethods,
-  patchClass: patchClass
+  patchClass: patchClass,
+  isWebWorker: isWebWorker
 };

Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,10 @@`
`1`	`1`	`'use strict';`
`2`	`2`
`3`	`3`	`var _redefineProperty = require('./define-property')._redefineProperty;`
	`4`	`+var utils = require("../utils");`
`4`	`5`
`5`	`6`	`function apply() {`
`6`		`- if (!('registerElement' in global.document)) {`
	`7`	`+ if (utils.isWebWorker() \|\| !('registerElement' in global.document)) {`
`7`	`8`	`return;`
`8`	`9`	`}`
`9`	`10`