Skip to content

Commit

Permalink
Fix #11443 #11441, shut up Chrome 19 and Safari 5.0.x. Closes gh-790.
Browse files Browse the repository at this point in the history
checkClone and appendChecked test failures in Chrome 19; reliableMarginRight failures in Safari 5.
  • Loading branch information
mikesherov authored and dmethvin committed May 22, 2012
1 parent e0151e5 commit 551f851
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 16 deletions.
18 changes: 5 additions & 13 deletions src/support.js
Expand Up @@ -179,7 +179,8 @@ jQuery.support = (function() {

// Run tests that need a body at doc ready
jQuery(function() {
var container, offsetSupport, marginDiv,
var container, marginDiv,
divReset = "padding:0;margin:0;border:0;display:block;overflow:hidden;",
conMarginTop = 1,
boxSizingPrefixes = [ "", "-moz-", "-webkit-", "" ],
body = document.getElementsByTagName("body")[0];
Expand Down Expand Up @@ -223,7 +224,7 @@ jQuery.support = (function() {
if ( window.getComputedStyle ) {
div.innerHTML = "";
marginDiv = document.createElement( "div" );
marginDiv.style.width = "0";
marginDiv.style.cssText = div.style.cssText = divReset;
marginDiv.style.marginRight = "0";
div.style.width = "2px";
div.appendChild( marginDiv );
Expand All @@ -237,11 +238,7 @@ jQuery.support = (function() {
// them layout
// (IE < 8 does this)
div.innerHTML = "";
div.style.width = div.style.padding = "1px";
div.style.border = 0;
div.style.overflow = "hidden";
div.style.display = "inline";
div.style.zoom = 1;
div.style.cssText = divReset + "width:1px;padding:1px;display:inline;zoom:1";
support.inlineBlockNeedsLayout = ( div.offsetWidth === 3 );

// Check if elements with layout shrink-wrap their children
Expand All @@ -259,19 +256,14 @@ jQuery.support = (function() {
support.pixelMargin = ( window.getComputedStyle( div, null ) || { marginTop: 0 } ).marginTop !== "1%";
}

offsetSupport = {
doesNotIncludeMarginInBodyOffset: ( body.offsetTop !== conMarginTop )
};

support.doesNotIncludeMarginInBodyOffset = ( body.offsetTop !== conMarginTop );

if ( typeof container.style.zoom !== "undefined" ) {
container.style.zoom = 1;
}

body.removeChild( container );
marginDiv = div = container = null;

jQuery.extend( support, offsetSupport );
});

return support;
Expand Down
6 changes: 3 additions & 3 deletions test/unit/support.js
Expand Up @@ -40,7 +40,7 @@ var userAgent = window.navigator.userAgent;

// These tests do not have to stay
// They are here to help with upcoming support changes for 1.8
if ( /chrome\/16\.0/i.test(userAgent) ) {
if ( /chrome\/19\.0/i.test(userAgent) ) {
test("Verify that the support tests resolve as expected per browser", function() {
var i,
expected = {
Expand Down Expand Up @@ -223,8 +223,8 @@ if ( /chrome\/16\.0/i.test(userAgent) ) {
"noCloneChecked":true,
"optDisabled":true,
"radioValue":true,
"checkClone":true,
"appendChecked":true,
"checkClone":false,
"appendChecked":false,
"boxModel":true,
"reliableHiddenOffsets":true,
"ajax":true,
Expand Down

0 comments on commit 551f851

Please sign in to comment.