Rewrite BlockBindingTransfomer for better performance

This is my proposition to optimize the BlockBindingTransfomer

The general idea behind it is:

* No longer use try-catch

* In variable declarations, transform any `let/const` to `var` and rename the variable to avoid name collisions.

* In loops, check if it contains any block bindings AND function declarations that use those block binded variables. 
If that's the case, then extract the loopBody to a function (~iife), and transform the final loopBody to invoke that function. 
The loopBody could have had `break`s, `continue`s and `return`s in it. These are handled by `if` statements and `switch` cases in the resulting transformed loopBody. Labels are accounted for (see tests)
* If the loop doesn't contain block bindings that are used in function declarations, then everything is transformed to `var` and renamed to avoid name collisions.

The PR isn't perfect and does have some rough edges. Please do comment on them and bash anything wrong that you see. It's my first contribution to traceur :-)
It does pass all the tests.

It's a significant change.
I'm free most days if anyone wants to pair up for quicker review
Hm, that should be handled in the DestructuringTransformer, no?
It even has [a comment about it](https://github.com/google/traceur-compiler/blob/5b3a2dbfec7fc00b2cc0c20eb6668a8366d49aca/src/codegeneration/FromOptionsTransformer.js#L166-L167)
>// destructuring must come after for of and before block binding and
    // generator

So I'm guessing what you want me to do is to have `getVariableName_` report an error if it finds a Pattern instead of a variableDeclaration?