Clean up jshint warnings in ./automation-tests

I like jshint, but opinions may vary. Anyways, this cleans up jshint warnings in ./automation-tests.

- it grandfathers use of 'foo["bar"]' since we use a lot of it
- cleans up a bunch of whiny stuff about semicolons
- removes some dead code jshint pointed out
- fix a couple of leaks to global of variables.
- prefers strict equality tests

I don't see any significant difference in test pass/fail rates on chrome, ff{win,linux}, ie9 and ie10.