A Test which indicates that the gHorizontalPercentageRegExp Regexp is…

… too greedy and a fix for the pattern which might not be optimal

See http://core.trac.wordpress.org/ticket/26181

lib/cssjanus.js

@@ -68,7 +68,7 @@ function CSSJanus() {
 		cursorWestRegExp = new RegExp( nonLetterPattern + '([ns]?)w-resize', 'gi' ),
 		fourNotationQuantRegExp = new RegExp( fourNotationQuantPropsPattern + signedQuantPattern + '(\\s+)' + signedQuantPattern + '(\\s+)' + signedQuantPattern + '(\\s+)' + signedQuantPattern, 'gi' ),
 		fourNotationColorRegExp = new RegExp( fourNotationColorPropsPattern + colorPattern + '(\\s+)' + colorPattern + '(\\s+)' + colorPattern + '(\\s+)' + colorPattern, 'gi' ),
-		bgHorizontalPercentageRegExp = new RegExp( '(background(?:-position)?\\s*:\\s*[^%]*?)(-?' + numPattern + ')(%\\s*(?:' + quantPattern + '|' + identPattern + '))', 'gi' ),
+		bgHorizontalPercentageRegExp = new RegExp( '(background(?:-position)?\\s*:\\s*[^%:;}]*?)(-?' + numPattern + ')(%\\s*(?:' + quantPattern + '|' + identPattern + '))', 'gi' ),
 		bgHorizontalPercentageXRegExp = new RegExp( '(background-position-x\\s*:\\s*)(-?' + numPattern + ')(%)', 'gi' ),
 		borderRadiusRegExp = new RegExp( '(border-radius\\s*:\\s*)([^;]*)', 'gi' );
 

test/cssjanus.test.js

@@ -400,6 +400,15 @@ var tests = {
 			flip( '.foo { background: 100% 20% } .bar { background: 60% 30% }' ),
 			'.foo { background: 0% 20% } .bar { background: 40% 30% }'
 		);
+		// Test that it doesn't modify other styles
+		assert.equal(
+			flip( '.foo { background: #777 } .bar{ margin: 0 5% 4% 0 }' ),
+			'.foo { background: #777 } .bar{ margin: 0 0 4% 5% }'
+		);
+		assert.equal(
+			flip( '.foo { background: #777; margin: 0 5% 4% 0 }' ),
+			'.foo { background: #777; margin: 0 0 4% 5% }'
+		);
 	},
 	'leaves class names alone': function() {
 		// Makes sure we don't unnecessarily destroy classnames with tokens in them.