Fixes #157

Rather than using SourceNodeMock objects when source maps are
disabled, use raw strings for better performance.
Cleanup.

* rename toSourceNode() to toSourceNodeWhenNeeded()
* clean up loop variables in flattenString()
Merge pull request #164 from msridhar/improve-performance

Patch to address #157