mpulse.js code review

[cvazac]

line 102, should we attach to onreadystatechange if typeof callback === "function"?
line 177, make sure Date is on window
line 273, typo "lenth"
line 580, if sessionID can be 0, then sessionID != null
line 586, should TODOs ship?
line 633, typo Stars
line 662: in stopTimer and sendTimer should degenerate case return something non-zero, maybe -1?
line 863, in subscribe, maybe ensure typeof callback === function
line 904, do test hooks ship?

[nicjansma]

Awesome, fixes in cbd21fb.

line 177, make sure Date is on window

Date is available on the root scope in NodeJS (not just window), so I just did an existence check.

line 586, should TODOs ship?

Fixing those depend on the JavaScript API collector changes on the schedule.

line 904, do test hooks ship?

I think so, I don't think it'll necessarily hurt. We could possibly build w/out that code block in Grunt.

[nicjansma]

All feedback was resolved.