[fixed] Refactored URLStore into multiple implementations

Mostly just cut/paste code into various locations handlers.  A nice side-effect is that we could use the `memoryHandler` for rendering on the server-side I would imagine.  Just need to support passing in the `currentPath` somehow.

All tests should be passing.  As @mjackson warned me, I did have some trouble with the tests, but they should all be passing now, and I think the coverage isn't too bad.

This location handler API will make it easy to plug in HistoryJS support now.

closes #166
Lazy lazy lazy. My bad. Let me fix those, make that change, run tests and
push back up.

Thanks,
-Sean

Sent from my iPhone

On Aug 5, 2014, at 10:25 PM, Michael Jackson <notifications@github.com>
wrote:

In modules/stores/URLStore.js:

> -
> -var _location;
> -var _currentPath = '/';
> -var _lastPath = null;
> -
> -function getWindowChangeEvent(location) {
> -  if (location === 'history')
> -    return 'popstate';
> -
> -  return window.addEventListener ? 'hashchange' : 'onhashchange';
> -}
> -
> -function getWindowPath() {
> -  return window.location.pathname + window.location.search;
> -}
> +var locationHandlers = require('../helpers/location/handlers');

This require path needs to be updated.

—
Reply to this email directly or view it on GitHub
<https://github.com/rackt/react-router/pull/169/files#r15856572>.