documentcloud/backbone — comment on commit 1332246b8f3b6672d3c3f19dd21c99765f4f8282
File:  line 0
By timcharper:

Thank you for merging this.

A few questions:

* Why is trigger: true usually undesirable? I almost always set that to true (and sometimes wish it were the default)
* Why did you remove the waitFor helper and replace it with the inferior setTimeout 500? setTimeout approach has the following disadvantages:
- It is coupled to a dependency that may not be present across all browsers: that document.location is updated within 500ms of a back invocation
- For browsers that take much, much less than 500ms to propagate said state, all runners of the test must pay a 500ms runtime penalty.

Anyways, it doesn't matter that much. Thanks again!