Style cleanup in server router.
Get rid of `this.stop()` !!
Correcting outdated examples in docs.md

onBeforeAction no longer uses call to this.stop() but uses pause() instead.
pathFor helper no longer accepts query string in format key=value but instead takes query='key=value'
Conflicts:
	DOCS.md
Router.configure({supressWarnings: true}) prevents warnings.
Supress warnings during testing.
Fix test to use newController vs legacy getController.
Update README.md
Update DOCS.md
Add semicolon

In the "declaring a route" example.
Update route_controller.js
Don’t call unnecessary stop() in client router when browsing away.

For #471
Added test case and fixed #676
Added test to prove I was wrong in #689
fix: ie9 omits leading slash in pathname on click handler

From https://github.com/EventedMind/iron-router/pull/650.

Thanks @schnie!
Added test and fix for #652
Make sure we clear unused regions if the before hook pauses.

And a test for #692. 

@cmather — I’m right in not doing this if the route is stopped right?
Make basic example a little more realistic.

Will be using this for the iron-layout migration.
Finish iron-layout integration.

@tmeasday,

Want to do a sanity check on this and make sure IR still works for you? The
expectation is that it should work as it did before (maybe with a few bugs
fixed) but uses the new iron-layout under the hood. I think if this works okay
we can do a patch release.

Next up I'll work on the larger refactor. I'll try to do this in stages so that
we can have a clear discussion around each thing and separate R&D type
improvemenets from general cleanup and API hardening.
wip - fix integrate iron-layout...

last step before trying to go to one computation again.
fix clearUnusedRegions bug and add waitOn test example

This is to make it work with the new Iron.Layout. Also note that the _run method
has been reduced to just one computation instead of many. This will all be
majorly overhauled and fixed up in the next release.