]> source.dussan.org Git - jquery.git/commitdiff
Ajax, Effects: Disable Chrome 31 workarounds
authorMichał Gołębiowski <m.goleb@gmail.com>
Mon, 10 Mar 2014 14:03:03 +0000 (15:03 +0100)
committerMichał Gołębiowski <m.goleb@gmail.com>
Mon, 10 Mar 2014 14:03:03 +0000 (15:03 +0100)
test/unit/ajax.js
test/unit/effects.js

index 4b4e408a3d61af70ac6cbc6b4fb379db93289645..2f4bc04e5bf91e2ef2c7c3781c724f1ebab53cd7 100644 (file)
@@ -1586,17 +1586,10 @@ module( "ajax", {
                }
        });
 
-       // Support: Chrome 31.
-       // Chrome 31 doesn't fire Ajax requests in beforeunload event handler.
-       // There is no way for us to workaround it and it's been fixed in Chrome 32
-       // so let's just blacklist Chrome 31 as long as it's in TestSwarm.
-       // See https://code.google.com/p/chromium/issues/detail?id=321241
-       if ( navigator.userAgent.indexOf( " Chrome/31." ) === -1 ) {
-               testIframeWithCallback( "#14379 - jQuery.ajax() on unload", "ajax/onunload.html", function( status ) {
-                       expect( 1 );
-                       strictEqual( status, "success", "Request completed" );
-               });
-       }
+       testIframeWithCallback( "#14379 - jQuery.ajax() on unload", "ajax/onunload.html", function( status ) {
+               expect( 1 );
+               strictEqual( status, "success", "Request completed" );
+       });
 
 //----------- jQuery.ajaxPrefilter()
 
index 2c9cb13c4e143628da9957d50147a2b36902c9a8..ca075408369b56dfb970b41edda588dcd4f0f0dc 100644 (file)
@@ -395,32 +395,20 @@ test("animate table-row width/height", function() {
 });
 
 test("animate table-cell width/height", function() {
-       // Support: Chrome 31.
-       // Chrome 31 reports incorrect offsetWidth on a table cell with fixed width.
-       // This is fixed in Chrome 32 so let's just skip the failing test in Chrome 31.
-       // See https://code.google.com/p/chromium/issues/detail?id=290399
-       var displayMode, td,
-               chrome31 = navigator.userAgent.indexOf( " Chrome/31." ) !== -1;
-               
-       if (chrome31) {
-               expect(2);
-       } else {
-               expect(3);
-       }
+       expect(3);
 
-       td = jQuery("#table")
+       var displayMode,
+               td = jQuery( "#table" )
                        .attr({ "cellspacing": 0, "cellpadding": 0, "border": 0 })
-                       .html("<tr><td style='width:42px;height:42px;padding:0;'><div style='width:20px;height:20px;'></div></td></tr>")
-                       .find("td");
+                       .html( "<tr><td style='width:42px;height:42px;padding:0;'><div style='width:20px;height:20px;'></div></td></tr>" )
+                       .find( "td" );
 
        // IE<8 uses "block" instead of the correct display type
-       displayMode = td.css("display") !== "table-cell" ? "block" : "table-cell";
+       displayMode = td.css( "display" ) !== "table-cell" ? "block" : "table-cell";
 
        td.animate({ width: 10, height: 10 }, 100, function() {
                equal( jQuery(this).css("display"), displayMode, "display mode is correct" );
-               if (!chrome31) {
-                       equal( this.offsetWidth, 20, "width animated to shrink wrap point" );
-               }
+               equal( this.offsetWidth, 20, "width animated to shrink wrap point" );
                equal( this.offsetHeight, 20, "height animated to shrink wrap point" );
        });
        this.clock.tick( 100 );