From 98da6b13bcc77dc117c68686ef4ed305bd83d400 Mon Sep 17 00:00:00 2001
From: timmywil <tim.willison@thisismedium.com>
Date: Mon, 11 Apr 2011 23:59:12 -0400
Subject: [PATCH] Fixes unit/css.js test fail in Safari 5. support.js test was
 passing erroneously due to the body to which the div was attached having a
 width of 0

---
 src/support.js   | 9 ++++++---
 test/unit/css.js | 2 +-
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/support.js b/src/support.js
index 34960505..6b19c083 100644
--- a/src/support.js
+++ b/src/support.js
@@ -8,6 +8,7 @@ jQuery.support = (function() {
 		select,
 		opt,
 		input,
+		marginDiv,
 		support,
 		fragment,
 		body,
@@ -190,10 +191,12 @@ jQuery.support = (function() {
 	// Fails in WebKit before Feb 2011 nightlies
 	// WebKit Bug 13343 - getComputedStyle returns wrong value for margin-right
 	if ( document.defaultView && document.defaultView.getComputedStyle ) {
-		div.style.width = "1px";
-		div.style.marginRight = "0";
+		marginDiv = document.createElement('div');
+		marginDiv.style.width = "0";
+		marginDiv.style.marginRight = "0";
+		div.appendChild( marginDiv );
 		support.reliableMarginRight =
-			( parseInt( document.defaultView.getComputedStyle(div).marginRight, 10 ) || 0 ) === 0;
+			( parseInt( document.defaultView.getComputedStyle( marginDiv ).marginRight, 10 ) || 0 ) === 0;
 	}
 
 	// Remove the body element we added
diff --git a/test/unit/css.js b/test/unit/css.js
index 08f50ef2..b94c6a8d 100644
--- a/test/unit/css.js
+++ b/test/unit/css.js
@@ -375,5 +375,5 @@ test("marginRight computed style (bug #3333)", function() {
 		marginRight: 0
 	});
 
-	equals($div.css("marginRight"), "0px");
+	equals($div.css("marginRight"), "0px", "marginRight correctly calculated with a width and display block");
 });