Use rhino properly, so that this works on openjdk
Fix up plovr tests
There's no good reason to access compiler Result directly
Use closure-compiler's error-handling, instead of re-inventing the wheel