From: Michał Gołębiowski <m.goleb@gmail.com>
Date: Fri, 6 Sep 2013 16:29:53 +0000 (+0200)
Subject: No ticket. Optimize top/left hooks: move the position check up, don't re-check it... 
X-Git-Tag: 1.11.0-beta1~51
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=6987220d2b561d73ad7c1f92988ffff239f8136b;p=jquery.git

No ticket. Optimize top/left hooks: move the position check up, don't re-check it inside a hook.
---

diff --git a/src/css.js b/src/css.js
index 660ed7df6..e40342d2d 100644
--- a/src/css.js
+++ b/src/css.js
@@ -589,22 +589,20 @@ jQuery.each( [ "top", "left" ], function( i, prop ) {
 				return;
 			}
 
-			if ( pixelPosition ) {
-				// Hook not needed, remove it.
+			if ( pixelPosition || !jQuery.fn.position ) {
+				// Hook not needed or impossible to apply due to a missing module, remove it.
 				// Since there are no other hooks for prop, remove the whole object.
 				delete jQuery.cssHooks[ prop ];
 				return;
 			}
 
 			jQuery.cssHooks[ prop ].get = function ( elem, computed ) {
-				if ( !support.pixelPosition() && jQuery.fn.position ) {
-					if ( computed ) {
-						computed = curCSS( elem, prop );
-						// if curCSS returns percentage, fallback to offset
-						return rnumnonpx.test( computed ) ?
-							jQuery( elem ).position()[ prop ] + "px" :
-							computed;
-					}
+				if ( computed ) {
+					computed = curCSS( elem, prop );
+					// if curCSS returns percentage, fallback to offset
+					return rnumnonpx.test( computed ) ?
+						jQuery( elem ).position()[ prop ] + "px" :
+						computed;
 				}
 			};