From 2ab6f18cee598383297597d14c9e21953fe0c1f7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20=C3=96berg?= Date: Wed, 9 Dec 2015 11:04:37 +0100 Subject: [PATCH] Changed isInIframe to a variable for caching --- src/mousetracker.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/mousetracker.js b/src/mousetracker.js index 425b22e7..1b26fab8 100644 --- a/src/mousetracker.js +++ b/src/mousetracker.js @@ -1367,7 +1367,7 @@ eventParams = getCaptureEventParams( tracker, $.MouseTracker.havePointerEvents ? 'pointerevent' : pointerType ); // We emulate mouse capture by hanging listeners on the document object. // (Note we listen on the capture phase so the captured handlers will get called first) - if (isInIframe() && canAccessEvents(window.top)) { + if (isInIframe && canAccessEvents(window.top)) { $.addEvent( window.top, eventParams.upName, @@ -1410,7 +1410,7 @@ eventParams = getCaptureEventParams( tracker, $.MouseTracker.havePointerEvents ? 'pointerevent' : pointerType ); // We emulate mouse capture by hanging listeners on the document object. // (Note we listen on the capture phase so the captured handlers will get called first) - if (isInIframe() && canAccessEvents(window.top)) { + if (isInIframe && canAccessEvents(window.top)) { $.removeEvent( window.top, eventParams.upName, @@ -3266,19 +3266,19 @@ } /** - * @function + * True if inside an iframe, otherwise false. + * @member {Boolean} isInIframe * @private * @inner - * @returns {Boolean} True if inside an iframe, otherwise false. */ - function isInIframe () { + var isInIframe = (function() { try { return window.self !== window.top; } catch (e) { return true; } - } - + })(); + /** * @function * @private