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

test/codegen_test.dart

Index: test/codegen_test.dart
diff --git a/test/codegen_test.dart b/test/codegen_test.dart
index 70f5a23f139a85413716e5e1ccbe4e000709916a..ba4c447028f05228ed64c72b18e2d830fb19a089 100644
--- a/test/codegen_test.dart
+++ b/test/codegen_test.dart
@@ -67,6 +67,7 @@ main(arguments) {
   BatchCompiler createCompiler(AnalysisContext context,
       {bool checkSdk: false,
       bool sourceMaps: false,
+      bool destructureNamedParams: false,
       bool closure: false,
       ModuleFormat moduleFormat: ModuleFormat.legacy}) {
     // TODO(jmesserly): add a way to specify flags in the test file, so
@@ -77,6 +78,7 @@ main(arguments) {
             outputDir: expectDir,
             emitSourceMaps: sourceMaps,
             closure: closure,
+            destructureNamedParams: destructureNamedParams,
             forceCompile: checkSdk,
             moduleFormat: moduleFormat),
         useColors: false,
@@ -157,6 +159,7 @@ $compilerMessages''';
           // We need a more comprehensive strategy to test them.
           var sourceMaps = filename == 'map_keys';
           var closure = filename == 'closure';
+          var destructureNamedParams = filename == 'destructuring' || closure;
           var moduleFormat = filename == 'es6_modules'
               ? ModuleFormat.es6
               : filename == 'node_modules'
@@ -166,13 +169,16 @@ $compilerMessages''';
           // TODO(vsm): Is it okay to reuse the same context here?  If there is
           // overlap between test files, we may need separate ones for each
           // compiler.
-          var compiler =
-              (sourceMaps || closure || moduleFormat != ModuleFormat.legacy)
-                  ? createCompiler(realSdkContext,
-                      sourceMaps: sourceMaps,
-                      closure: closure,
-                      moduleFormat: moduleFormat)
-                  : batchCompiler;
+          var compiler = (sourceMaps ||
+                  closure ||
+                  destructureNamedParams ||
+                  moduleFormat != ModuleFormat.legacy)
+              ? createCompiler(realSdkContext,
+                  sourceMaps: sourceMaps,
+                  destructureNamedParams: destructureNamedParams,
+                  closure: closure,
+                  moduleFormat: moduleFormat)
+              : batchCompiler;
           success = compile(compiler, filePath);
 
           var outFile = new File(path.join(outDir.path, '$filename.js'));