Refactor location module.

This commit is contained in:
Lars Jung 2015-04-29 00:44:10 +02:00
parent 2525ad8393
commit 6500b624f5
2 changed files with 11 additions and 11 deletions

View file

@ -178,17 +178,17 @@ modulejs.define('core/location', ['_', 'modernizr', 'core/event', 'core/notify',
}
}
if (history) {
window.onpopstate = function (ev) {
function onPopState(ev) {
if (ev.state && ev.state.absHref) {
setLocation(ev.state.absHref, true);
}
};
}
window.onpopstate = history ? onPopState : null;
return {
forceEncoding: forceEncoding,
getDomain: getDomain,

View file

@ -86,7 +86,7 @@ describe('module \'' + ID + '\'', function () {
assert.lengthOfKeys(instance, 7);
});
it('sets window.onpopstate when history and fastBrowsing', function () {
it('sets window.onpopstate function when history and fastBrowsing', function () {
this.xModernizr.history = true;
this.xSettings.view.fastBrowsing = true;
@ -96,7 +96,7 @@ describe('module \'' + ID + '\'', function () {
assert.isFunction(window.onpopstate);
});
it('does not set window.onpopstate when not history and fastBrowsing', function () {
it('sets window.onpopstate to null when not history and fastBrowsing', function () {
this.xModernizr.history = false;
this.xSettings.view.fastBrowsing = true;
@ -106,7 +106,7 @@ describe('module \'' + ID + '\'', function () {
assert.isNull(window.onpopstate);
});
it('does not set window.onpopstate when history and not fastBrowsing', function () {
it('sets window.onpopstate to null when history and not fastBrowsing', function () {
this.xModernizr.history = true;
this.xSettings.view.fastBrowsing = false;
@ -116,7 +116,7 @@ describe('module \'' + ID + '\'', function () {
assert.isNull(window.onpopstate);
});
it('does not set window.onpopstate when not history and not fastBrowsing', function () {
it('sets window.onpopstate to null when not history and not fastBrowsing', function () {
this.xModernizr.history = false;
this.xSettings.view.fastBrowsing = false;