aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoronnoj <me@onnoj.net>2014-01-02 11:57:21 +0100
committeronnoj <me@onnoj.net>2014-01-02 11:57:21 +0100
commit76ba593a1cbd2f55c1bb8966929405b4a47a3185 (patch)
treef371d780ad9f1d9bc5b11e8cea9c3f83c37d4c23
parente132e7a2b54564ffb710c4fc5a8f29fc0431f847 (diff)
Fixed undefined check.
Forgot I had to explicitly check the type of variable type to see if it's undefined.
-rw-r--r--src/library_browser.js6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/library_browser.js b/src/library_browser.js
index bf8444bd..029598f3 100644
--- a/src/library_browser.js
+++ b/src/library_browser.js
@@ -482,12 +482,12 @@ mergeInto(LibraryManager.library, {
//Neither .scrollX or .pageXOffset are defined in a spec, but
//we prefer .scrollX because it is currently in a spec draft.
//(see: http://www.w3.org/TR/2013/WD-cssom-view-20131217/)
- var scrollX = ((window.scrollX !== undefined) ? window.scrollX : window.pageXOffset);
- var scrollY = ((window.scrollY !== undefined) ? window.scrollY : window.pageYOffset);
+ var scrollX = ((typeof window.scrollX !== 'undefined') ? window.scrollX : window.pageXOffset);
+ var scrollY = ((typeof window.scrollY !== 'undefined') ? window.scrollY : window.pageYOffset);
#if ASSERTIONS
//If this assert lands, it's likely because the browser doesn't support scrollX or pageXOffset
//and we have no viable fallback.
- assert((scrollX !== undefined) && (scrollY !== undefined), 'Unable to retrieve scroll position, mouse positions likely broken.');
+ assert((typeof scrollX !== 'undefined') && (typeof scrollY !== 'undefined'), 'Unable to retrieve scroll position, mouse positions likely broken.');
#endif
if (event.type == 'touchstart' ||
event.type == 'touchend' ||