Hook up the --minify option in dart2js and use it to get rid of unnecessary whitespace inside funct…

lib/compiler/implementation/dart_backend/renamer.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.
 
 /**
  * Renames only top-level elements that would let to ambiguity if not renamed.
  * TODO(smok): Make sure that top-level fields are correctly renamed.
  */
 void renamePlaceholders(
     Compiler compiler,
     PlaceholderCollector placeholderCollector,
     Map<Node, String> renames,
     Map<LibraryElement, String> imports,
     Set<String> fixedMemberNames,
-    bool minify,
     bool cutDeclarationTypes) {
   final Map<LibraryElement, Map<String, String>> renamed
       = new Map<LibraryElement, Map<String, String>>();
-  Generator topLevelGenerator =
-      minify ? new MinifyingGenerator('ABCDEFGHIJKLMNOPQRSTUVWXYZ').generate
-          : conservativeGenerator;
+  Generator topLevelGenerator = compiler.enableMinification
+      ? new MinifyingGenerator('ABCDEFGHIJKLMNOPQRSTUVWXYZ').generate
+      : conservativeGenerator;
   makeGenerator(usedIdentifierSet) => (name) {
     String newName = topLevelGenerator(name, usedIdentifierSet.contains);
     usedIdentifierSet.add(newName);
     return newName;
   };
 
   final usedTopLevelIdentifiers = new Set<String>();
   // TODO(antonm): we should also populate this set with top-level
   // names from core library.
   usedTopLevelIdentifiers.add('main'); // Never rename anything to 'main'.
@@ skipping 37 matching lines @@
   renameNodes(placeholderCollector.nullNodes, (_) => '');
   renameNodes(placeholderCollector.unresolvedNodes,
       (_) => generateUniqueName('Unresolved'));
   sortedForEach(placeholderCollector.elementNodes, (element, nodes) {
     String renamedElement = renameElement(element);
     renameNodes(nodes, (_) => renamedElement);
   });
   sortedForEach(placeholderCollector.functionScopes,
       (functionElement, functionScope) {
     Set<LocalPlaceholder> placeholders = functionScope.localPlaceholders;
-    Generator localGenerator =
-        minify ? new MinifyingGenerator('abcdefghijklmnopqrstuvwxyz').generate
-            : conservativeGenerator;
+    Generator localGenerator = compiler.enableMinification
+        ? new MinifyingGenerator('abcdefghijklmnopqrstuvwxyz').generate
+        : conservativeGenerator;
     Set<String> memberIdentifiers = new Set<String>();
     if (functionElement.getEnclosingClass() !== null) {
       functionElement.getEnclosingClass().forEachMember(
           (enclosingClass, member) {
         memberIdentifiers.add(member.name.slowToString());
       });
     }
     Set<String> usedLocalIdentifiers = new Set<String>();
     // TODO(smok): Take usages into account.
     for (LocalPlaceholder placeholder in placeholders) {
@@ skipping 67 matching lines @@
     int length = alphabet.length;
     StringBuffer resultBuilder = new StringBuffer();
     while (index >= length) {
       resultBuilder.add(alphabet[index % length]);
       index ~/= length;
     }
     resultBuilder.add(alphabet[index]);
     return resultBuilder.toString();
   }
 }