fixed hot handling

it now bubbles when the css failed
fixes webpack-contrib#11
fixes webpack-contrib#12
fixes webpack-contrib#13

Author: sokra

addStyle.js

@@ -15,7 +15,17 @@ module.exports = function addStyle(cssCode) {
 	} else {
 		styleElement.appendChild(document.createTextNode(cssCode));
 	}
-	return function() {
-		head.removeChild(styleElement);
-	};
+	if(module.hot) {
+		return function(cssCode) {
+			if(typeof cssCode === "string") {
+				if (styleElement.styleSheet) {
+					styleElement.styleSheet.cssText = cssCode;
+				} else {
+					styleElement.childNodes[0].nodeValue = cssCode;
+				}
+			} else {
+				head.removeChild(styleElement);
+			}
+		};
+	}
 }

addStyleUrl.js

@@ -12,7 +12,13 @@ module.exports = function addStyleUrl(cssUrl) {
 	styleElement.href = cssUrl;
 	var head = document.getElementsByTagName("head")[0];
 	head.appendChild(styleElement);
-	return function() {
-		head.removeChild(styleElement);
-	};
+	if(module.hot) {
+		return function(cssUrl) {
+			if(typeof cssUrl === "string") {
+				styleElement.href = cssUrl;
+			} else {
+				head.removeChild(styleElement);
+			}
+		};
+	}
 }

index.js

@@ -6,11 +6,17 @@ var path = require("path");
 module.exports = function() {};
 module.exports.pitch = function(remainingRequest) {
 	this.cacheable && this.cacheable();
-	var comment1 = "// style-loader: Adds some css to the DOM by adding a <style> tag\n";
-	var addStyleCode = "var dispose = require(" + JSON.stringify("!" + path.join(__dirname, "addStyle.js")) + ")\n";
-	var comment2 = "\t// The css code:\n";
-	var cssCodeRequest = "require(" + JSON.stringify("!!" + remainingRequest) + ")";
-	var comment3 = "// Hot Module Replacement\n";
-	var hmrCode = "if(module.hot) {\n\tmodule.hot.accept();\n\tmodule.hot.dispose(dispose);\n}";
-	return comment1 + addStyleCode + comment2 + "\t(" + cssCodeRequest + ");\n" + comment3 + hmrCode;
+	return [
+		"// style-loader: Adds some css to the DOM by adding a <style> tag",
+		"var update = require(" + JSON.stringify("!" + path.join(__dirname, "addStyle.js")) + ")(",
+		"\trequire(" + JSON.stringify("!!" + remainingRequest) + ")",
+		");",
+		"// Hot Module Replacement",
+		"if(module.hot) {",
+		"\tmodule.hot.accept(" + JSON.stringify("!!" + remainingRequest) + ", function() {",
+		"\t\tupdate(require(" + JSON.stringify("!!" + remainingRequest) + "));",
+		"\t});",
+		"\tmodule.hot.dispose(function() { update(); });",
+		"}"
+	].join("\n");
 };

url.js

@@ -6,11 +6,17 @@ var path = require("path");
 module.exports = function() {};
 module.exports.pitch = function(remainingRequest) {
 	this.cacheable && this.cacheable();
-	var comment1 = "// style-loader: Adds some css to the DOM by adding a <style> tag\n";
-	var addStyleCode = "var dispose = require(" + JSON.stringify("!" + path.join(__dirname, "addStyleUrl.js")) + ")\n";
-	var comment2 = "\t// The url to the css code:\n";
-	var cssUrlRequest = "require(" + JSON.stringify("!!" + remainingRequest) + ")";
-	var comment3 = "// Hot Module Replacement\n";
-	var hmrCode = "if(module.hot) {\n\tmodule.hot.accept();\n\tmodule.hot.dispose(dispose);\n}";
-	return comment1 + addStyleCode + comment2 + "\t(" + cssUrlRequest + ");\n" + comment3 + hmrCode;
+	return [
+		"// style-loader: Adds some reference to a css file to the DOM by adding a <link> tag",
+		"var update = require(" + JSON.stringify("!" + path.join(__dirname, "addStyleUrl.js")) + ")(",
+		"\trequire(" + JSON.stringify("!!" + remainingRequest) + ")",
+		");",
+		"// Hot Module Replacement",
+		"if(module.hot) {",
+		"\tmodule.hot.accept(" + JSON.stringify("!!" + remainingRequest) + ", function() {",
+		"\t\tupdate(require(" + JSON.stringify("!!" + remainingRequest) + "));",
+		"\t});",
+		"\tmodule.hot.dispose(function() { update(); });",
+		"}"
+	].join("\n");
 };