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 d9104ad3f702425bd60f7222fb50d1dcbb685a65..ec55a24115a9ee8181459f33873f35b46370cc6b 100644 (file)
--- 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 71c82e9935acc3a3381fe479ee77b041a15e8d58..c2213b248266be2c8770d6a00c203f35e6d6b96e 100644 (file)
--- 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 ) {