Use default params when --destructure-named-params + fix renaming of reserved destructured params

lib/src/js/printer.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 part of js_ast;
 
 
 class JavaScriptPrintingOptions {
   final bool shouldCompressOutput;
   final bool minifyLocalVariables;
@@ skipping 603 matching lines @@
     out("]");
   }
   visitObjectBindingPattern(ObjectBindingPattern node) {
     out("{");
     visitCommaSeparated(node.variables, EXPRESSION,
                         newInForInit: false, newAtStatementBegin: false);
     out("}");
   }
 
   visitDestructuredVariable(DestructuredVariable node) {
-    var hasName = node.name != null;
-    if (hasName) visit(node.name);
+    var name = node.name;
+    var hasName = name != null;
+    if (hasName) {
+      if (name is LiteralString) {
+        out("[");
+        out(name.value);
+        out("]");
+      } else {
+        visit(name);
+      }
+    }
     if (node.structure != null) {
       if (hasName) {
         out(":");
         spaceOut();
       }
       visit(node.structure);
     }
     if (node.defaultValue != null) {
       spaceOut();
       out("=");
       spaceOut();
       visitNestedExpression(node.defaultValue, EXPRESSION,
                             newInForInit: false, newAtStatementBegin: false);
     }
   }
 
+  visitSimpleBindingPattern(SimpleBindingPattern node) {
+    visit(node.name);
+  }
+
   visitAssignment(Assignment assignment) {
     visitNestedExpression(assignment.leftHandSide, LEFT_HAND_SIDE,
                           newInForInit: inForInit,
                           newAtStatementBegin: atStatementBegin);
     if (assignment.value != null) {
       spaceOut();
       String op = assignment.op;
       if (op != null) out(op);
       out("=");
       spaceOut();
@@ skipping 926 matching lines @@
 abstract class VariableDeclarationVisitor<T> extends BaseVisitor<T> {
   declare(Identifier node);
 
   visitFunctionExpression(FunctionExpression node) {
     node.params.forEach(_scanVariableBinding);
     node.body.accept(this);
   }
 
   _scanVariableBinding(VariableBinding d) {
     if (d is Identifier) declare(d);
-    if (d is RestParameter) _scanVariableBinding(d.parameter);
-    else if (d is BindingPattern) {
-      for (var v in d.variables) {
-        if (v.name != null) declare(v.name);
-        if (v.structure != null) _scanVariableBinding(v.structure);
-      }
-    }
+    else d.accept(this);
+  }
+
+  visitRestParameter(RestParameter node) {
+    _scanVariableBinding(node.parameter);
+    super.visitRestParameter(node);
+  }
+
+  visitDestructuredVariable(DestructuredVariable node) {
+    var name = node.name;
+    if (name is Identifier) _scanVariableBinding(name);
+    super.visitDestructuredVariable(node);
+  }
+
+  visitSimpleBindingPattern(SimpleBindingPattern node) {
+    _scanVariableBinding(node.name);
+    super.visitSimpleBindingPattern(node);
   }
 
   visitVariableInitialization(VariableInitialization node) {
     _scanVariableBinding(node.declaration);
     if (node.value != null) node.value.accept(this);
   }
 
   visitCatch(Catch node) {
     declare(node.declaration);
     node.body.accept(this);
   }
 
   visitFunctionDeclaration(FunctionDeclaration node) {
     declare(node.name);
     node.function.accept(this);
   }
 
   visitNamedFunction(NamedFunction node) {
     declare(node.name);
     node.function.accept(this);
   }
 
   visitClassExpression(ClassExpression node) {
     declare(node.name);
     if (node.heritage != null) node.heritage.accept(this);
     for (Method element in node.methods) element.accept(this);
   }
 }