From: Jun Sun <klsforever@gmail.com>
Date: Wed, 30 Dec 2015 07:40:08 +0000 (+0800)
Subject: CSS: Add animation-iteration-count to cssNumber, fix tests
X-Git-Tag: 2.2.0~9
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b9a695865b4eb3916f7320026d028a2866da6e4e;p=jquery.git

CSS: Add animation-iteration-count to cssNumber, fix tests

Fixes gh-2792
Closes gh-2793
---

diff --git a/src/css.js b/src/css.js
index d9104ad3f..ec55a2411 100644
--- a/src/css.js
+++ b/src/css.js
@@ -252,6 +252,7 @@ jQuery.extend( {
 
 	// Don't automatically add "px" to these possibly-unitless properties
 	cssNumber: {
+		"animationIterationCount": true,
 		"columnCount": true,
 		"fillOpacity": true,
 		"flexGrow": true,
diff --git a/test/unit/css.js b/test/unit/css.js
index 71c82e993..c2213b248 100644
--- a/test/unit/css.js
+++ b/test/unit/css.js
@@ -848,26 +848,35 @@ if ( jQuery.fn.offset ) {
 	} );
 }
 
-QUnit.test( "Do not append px (#9548, #12990)", function( assert ) {
-	assert.expect( 2 );
+QUnit.test( "Do not append px (#9548, #12990, #2792)", function( assert ) {
+	assert.expect( 3 );
 
 	var $div = jQuery( "<div>" ).appendTo( "#qunit-fixture" );
 
 	$div.css( "fill-opacity", 1 );
 
 	// Support: Android 2.3 (no support for fill-opacity)
-	if ( $div.css( "fill-opacity" ) ) {
+	if ( $div.css( "fill-opacity" ) !== undefined ) {
 		assert.equal( $div.css( "fill-opacity" ), 1, "Do not append px to 'fill-opacity'" );
 	} else {
 		assert.ok( true, "No support for fill-opacity CSS property" );
 	}
 
 	$div.css( "column-count", 1 );
-	if ( $div.css( "column-count" ) ) {
+	if ( $div.css( "column-count" ) !== undefined ) {
 		assert.equal( $div.css( "column-count" ), 1, "Do not append px to 'column-count'" );
 	} else {
 		assert.ok( true, "No support for column-count CSS property" );
 	}
+
+	$div.css( "animation-iteration-count", 2 );
+	if ( $div.css( "animation-iteration-count" ) !== undefined ) {
+		// if $div.css( "animation-iteration-count" ) return "1",
+		// it actually return the default value of animation-iteration-count
+		assert.equal( $div.css( "animation-iteration-count" ), 2, "Do not append px to 'animation-iteration-count'" );
+	} else {
+		assert.ok( true, "No support for animation-iteration-count CSS property" );
+	}
 } );
 
 QUnit.test( "css('width') and css('height') should respect box-sizing, see #11004", function( assert ) {