jshint style tests

This is my proposal for reworking the style tests.  I am now install npm dependencies within cmake because we are relying on specific versions.  The diff for correcting all of the source files will be quite large.  The last commit of this PR fixes several of the core files as an example, but I will probably have to rebase that commit after the changes coming in from #127.  I'm going to recommend holding off on this until we can merge in all changes on the pipe line.
No jshint doesn't catch things like this.  At one point, I was changing all inner function references from `this` to `m_this`, but I surely didn't get them all.  We should make a point to do that in the future.