Convert webworker template runtime to Promise-based API

tsortwehttam · tsortwehttam · commit 2e8f08f6ac68 · 2016-09-12T18:40:23.000-04:00
Fixes 'cannot call method then on undefined' error when using HMR with a config whose target is "webworker". Probably should have happened with 2245c4a
diff --git a/lib/webworker/WebWorkerMainTemplate.runtime.js b/lib/webworker/WebWorkerMainTemplate.runtime.js
@@ -14,39 +14,41 @@ module.exports = function() {
 	}
 
 	function hotDownloadManifest(callback) { // eslint-disable-line no-unused-vars
-		if(typeof XMLHttpRequest === "undefined")
-			return callback(new Error("No browser support"));
-		try {
-			var request = new XMLHttpRequest();
-			var requestPath = $require$.p + $hotMainFilename$;
-			request.open("GET", requestPath, true);
-			request.timeout = 10000;
-			request.send(null);
-		} catch(err) {
-			return callback(err);
-		}
-		request.onreadystatechange = function() {
-			if(request.readyState !== 4) return;
-			if(request.status === 0) {
-				// timeout
-				callback(new Error("Manifest request to " + requestPath + " timed out."));
-			} else if(request.status === 404) {
-				// no update available
-				callback();
-			} else if(request.status !== 200 && request.status !== 304) {
-				// other failure
-				callback(new Error("Manifest request to " + requestPath + " failed."));
-			} else {
-				// success
-				try {
-					var update = JSON.parse(request.responseText);
-				} catch(e) {
-					callback(e);
-					return;
+		return new Promise(function(resolve, reject) {
+			if(typeof XMLHttpRequest === "undefined")
+				return reject(new Error("No browser support"));
+			try {
+				var request = new XMLHttpRequest();
+				var requestPath = $require$.p + $hotMainFilename$;
+				request.open("GET", requestPath, true);
+				request.timeout = 10000;
+				request.send(null);
+			} catch(err) {
+				return reject(err);
+			}
+			request.onreadystatechange = function() {
+				if(request.readyState !== 4) return;
+				if(request.status === 0) {
+					// timeout
+					reject(new Error("Manifest request to " + requestPath + " timed out."));
+				} else if(request.status === 404) {
+					// no update available
+					resolve();
+				} else if(request.status !== 200 && request.status !== 304) {
+					// other failure
+					reject(new Error("Manifest request to " + requestPath + " failed."));
+				} else {
+					// success
+					try {
+						var update = JSON.parse(request.responseText);
+					} catch(e) {
+						reject(e);
+						return;
+					}
+					resolve(update);
 				}
-				callback(null, update);
 			}
-		};
+		});
 	}
 
 	function hotDiposeChunk(chunkId) {
