Better Hash/QS support in router

Better Hash/QS support in router

This is a small update to bcsaller's original branch to add support for
non word characters in the hash key:
https://code.launchpad.net/~bcsaller/juju-gui/routing-hash-query/+merge/168351

previous comment:
`parse`, `url` and `combine` deal more gracefully with hash and qs url
options. In the case of combine the incoming url replaces any existing
values in the hash and qs portions of the URL as might be expected. (This
is rather than some artificial merge policy for example).

https://code.launchpad.net/~hatch/juju-gui/routing-hash-query/+merge/168555

(do not edit description out of merge proposal)

[jeff.pihach, 2013-06-10 22:14:25]

Please take a look.

[rharding, 2013-06-10 22:25:43]

LGTM with comment about following up on the TODO

https://codereview.appspot.com/10176043/diff/1/app/assets/javascripts/ns-rout...
File app/assets/javascripts/ns-routing-app-extension.js (right):

https://codereview.appspot.com/10176043/diff/1/app/assets/javascripts/ns-rout...
app/assets/javascripts/ns-routing-app-extension.js:348: url = this.url(output,
{excludeRootPaths: true});
Should there be a bug and an XXX: for the todo or is there a process to follow
up TODO in the code today?

[jeff.pihach, 2013-06-10 22:44:35]

*** Submitted:

Better Hash/QS support in router

This is a small update to bcsaller's original branch to add support for
non word characters in the hash key:
https://code.launchpad.net/~bcsaller/juju-gui/routing-hash-query/+merge/168351

previous comment:
`parse`, `url` and `combine` deal more gracefully with hash and qs url
options. In the case of combine the incoming url replaces any existing
values in the hash and qs portions of the URL as might be expected. (This
is rather than some artificial merge policy for example).

R=rharding
CC=
https://codereview.appspot.com/10176043