Offset: Eliminate little-used internal function

gibson042 · web-flow · commit bf3a43eff868 · 2016-12-19T12:00:46.000-05:00
Fixes gh-3449 Closes gh-3456
diff --git a/src/offset.js b/src/offset.js
@@ -15,13 +15,6 @@ define( [
 
 "use strict";
 
-/**
- * Gets a window from an element
- */
-function getWindow( elem ) {
-	return jQuery.isWindow( elem ) ? elem : elem.nodeType === 9 && elem.defaultView;
-}
-
 jQuery.offset = {
 	setOffset: function( elem, options, i ) {
 		var curPosition, curLeft, curCSSTop, curTop, curOffset, curCSSLeft, calculatePosition,
@@ -86,7 +79,7 @@ jQuery.fn.extend( {
 				} );
 		}
 
-		var docElem, win, rect, doc,
+		var doc, docElem, rect, win,
 			elem = this[ 0 ];
 
 		if ( !elem ) {
@@ -104,8 +97,8 @@ jQuery.fn.extend( {
 		rect = elem.getBoundingClientRect();
 
 		doc = elem.ownerDocument;
-		win = getWindow( doc );
 		docElem = doc.documentElement;
+		win = doc.defaultView;
 
 		return {
 			top: rect.top + win.pageYOffset - docElem.clientTop,
@@ -183,7 +176,14 @@ jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function(
 
 	jQuery.fn[ method ] = function( val ) {
 		return access( this, function( elem, method, val ) {
-			var win = getWindow( elem );
+
+			// Coalesce documents and windows
+			var win;
+			if ( jQuery.isWindow( elem ) ) {
+				win = elem;
+			} else if ( elem.nodeType === 9 ) {
+				win = elem.defaultView;
+			}
 
 			if ( val === undefined ) {
 				return win ? win[ prop ] : elem[ method ];
