dart2js OldEmitter: Small cleanups in output.

pkg/compiler/lib/src/js_emitter/old_emitter/reflection_data_parser.dart

 // Copyright (c) 2013, 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 dart2js.js_emitter;
 
 // TODO(ahe): Share these with js_helper.dart.
 const FUNCTION_INDEX = 0;
 const NAME_INDEX = 1;
 const CALL_NAME_INDEX = 2;
@@ skipping 216 matching lines @@
     var isAccessor = (requiredParameterInfo & 1) === 1;
     var isSetter = requiredParameterInfo === 3;
     var isGetter = requiredParameterInfo === 1;
     var optionalParameterInfo = ${readInt("array", "1")};
     var optionalParameterCount = optionalParameterInfo >> 1;
     var optionalParametersAreNamed = (optionalParameterInfo & 1) === 1;
     var isIntercepted =
            requiredParameterCount + optionalParameterCount != funcs[0].length;
     var functionTypeIndex = ${readFunctionType("array", "2")};
     var unmangledNameIndex = $unmangledNameIndex;
-    var isReflectable = array.length > unmangledNameIndex;
 
     if (getterStubName) {
       f = tearOff(funcs, array, isStatic, name, isIntercepted);
       descriptor[name].\$getter = f;
       f.\$getterStub = true;
       // Used to create an isolate using spawnFunction.
       if (isStatic) #globalFunctions[name] = f;
       originalDescriptor[getterStubName] = descriptor[getterStubName] = f;
       funcs.push(f);
       if (getterStubName) functions.push(getterStubName);
       f.\$stubName = getterStubName;
       f.\$callName = null;
       if (isIntercepted) #interceptedNames[getterStubName] = true;
     }
-    if (isReflectable) {
-      for (var i = 0; i < funcs.length; i++) {
-        funcs[i].$reflectableField = 1;
-        funcs[i].$reflectionInfoField = array;
+
+    if (#usesMangledNames) {
+      var isReflectable = array.length > unmangledNameIndex;
+      if (isReflectable) {
+        for (var i = 0; i < funcs.length; i++) {
+          funcs[i].$reflectableField = 1;
+          funcs[i].$reflectionInfoField = array;
+        }
+        var mangledNames = isStatic ? #mangledGlobalNames : #mangledNames;
+        var unmangledName = ${readString("array", "unmangledNameIndex")};
+        // The function is either a getter, a setter, or a method.
+        // If it is a method, it might also have a tear-off closure.
+        // The unmangledName is the same as the getter-name.
+        var reflectionName = unmangledName;
+        if (getterStubName) mangledNames[getterStubName] = reflectionName;
+        if (isSetter) {
+          reflectionName += "=";
+        } else if (!isGetter) {
+          reflectionName += ":" + requiredParameterCount +
+            ":" + optionalParameterCount;
+        }
+        mangledNames[name] = reflectionName;
+        funcs[0].$reflectionNameField = reflectionName;
+        funcs[0].$metadataIndexField = unmangledNameIndex + 1;
+        if (optionalParameterCount) descriptor[unmangledName + "*"] = funcs[0];
       }
-      var mangledNames = isStatic ? #mangledGlobalNames : #mangledNames;
-      var unmangledName = ${readString("array", "unmangledNameIndex")};
-      // The function is either a getter, a setter, or a method.
-      // If it is a method, it might also have a tear-off closure.
-      // The unmangledName is the same as the getter-name.
-      var reflectionName = unmangledName;
-      if (getterStubName) mangledNames[getterStubName] = reflectionName;
-      if (isSetter) {
-        reflectionName += "=";
-      } else if (!isGetter) {
-        reflectionName += ":" + requiredParameterCount +
-          ":" + optionalParameterCount;
-      }
-      mangledNames[name] = reflectionName;
-      funcs[0].$reflectionNameField = reflectionName;
-      funcs[0].$metadataIndexField = unmangledNameIndex + 1;
-      if (optionalParameterCount) descriptor[unmangledName + "*"] = funcs[0];
     }
   }
 ''', {'globalFunctions' : globalFunctionsAccess,
       'interceptedNames': interceptedNamesAccess,
+      'usesMangledNames':
+          compiler.mirrorsLibrary != null || compiler.enabledFunctionApply,
       'mangledGlobalNames': mangledGlobalNamesAccess,
       'mangledNames': mangledNamesAccess});
 
   List<jsAst.Statement> tearOffCode = buildTearOffCode(backend);
 
   jsAst.Statement init = js.statement('''{
   var functionCounter = 0;
   if (!#libraries) #libraries = [];
   if (!#mangledNames) #mangledNames = map();
   if (!#mangledGlobalNames) #mangledGlobalNames = map();
@@ skipping 274 matching lines @@
 (function() {
   var result = $array[$index];
   if ($check) {
     throw new Error(
         name + ": expected value of type \'$type\' at index " + ($index) +
         " but got " + (typeof result));
   }
   return result;
 })()''';
 }