Fixed: trim url first.

Author: evilebottnawi

lib/processCss.js

@@ -161,18 +161,18 @@ module.exports = function processCss(inputSource, inputMap, options, callback) {
 	var pipeline = postcss([
 		localByDefault({
 			mode: options.mode,
-			rewriteUrl: function(global, origUrl) {
+			rewriteUrl: function(global, url) {
 				if(parserOptions.url){
-					var url = origUrl.trim(" ");
+                    url = url.trim();
 
-					if(!origUrl.replace(/\s/g, '').length || !loaderUtils.isUrlRequest(url, root)) {
-						return origUrl;
+					if(!url.replace(/\s/g, '').length || !loaderUtils.isUrlRequest(url, root)) {
+						return url;
 					}
 					if(global) {
 						return loaderUtils.urlToRequest(url, root);
 					}
 				}
-				return origUrl;
+				return url;
 			}
 		}),
 		extractImports(),

test/urlTest.js

@@ -77,16 +77,16 @@ describe("url", function() {
 		[1, ".class { background: green url('') xyz }", ""]
 	]);
 	test("empty url with spaces and quotes", ".class { background: green url('   ') xyz }", [
-		[1, ".class { background: green url('   ') xyz }", ""]
+		[1, ".class { background: green url('') xyz }", ""]
 	]);
 	test("empty url with newline and quotes", ".class { background: green url('\n') xyz }", [
-		[1, ".class { background: green url('\n') xyz }", ""]
+		[1, ".class { background: green url('') xyz }", ""]
 	]);
 	test("empty url with CRLF and quotes", ".class { background: green url('\r\n') xyz }", [
-		[1, ".class { background: green url('\r\n') xyz }", ""]
+		[1, ".class { background: green url('') xyz }", ""]
 	]);
 	test("empty url with tab and quotes", ".class { background: green url('\t') xyz }", [
-		[1, ".class { background: green url('\t') xyz }", ""]
+		[1, ".class { background: green url('') xyz }", ""]
 	]);
 	test("external absolute url", ".class { background: green url(https://raw.githubusercontent.com/webpack/media/master/logo/icon.png) xyz }", [
 		[1, ".class { background: green url(https://raw.githubusercontent.com/webpack/media/master/logo/icon.png) xyz }", ""]