Merge pull request webpack#4607 from jun0205/webpack-1

sokra · web-flow · commit 8d5ed06733cb · 2017-04-01T00:09:38.000+02:00
IE8 for-in, skip prototype property.
diff --git a/lib/JsonpMainTemplatePlugin.js b/lib/JsonpMainTemplatePlugin.js
@@ -105,7 +105,11 @@ JsonpMainTemplatePlugin.prototype.apply = function(mainTemplate) {
 					]),
 					"}",
 					"for(moduleId in moreModules) {",
-					this.indent(this.renderAddModule(hash, chunk, "moduleId", "moreModules[moduleId]")),
+					this.indent([
+						"if(Object.prototype.hasOwnProperty.call(moreModules, moduleId)) {",
+						this.indent(this.renderAddModule(hash, chunk, "moduleId", "moreModules[moduleId]")),
+						"}"
+					]),
 					"}",
 					"if(parentJsonpFunction) parentJsonpFunction(chunkIds, moreModules);",
 					"while(callbacks.length)",
