Patch refactoring.

lib/compiler/implementation/dart_backend/backend.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.
 
 const bool REMOVE_ASSERTS = false;
 
 class ElementAst {
   final Node ast;
   final TreeElements treeElements;
 
@@ skipping 182 matching lines @@
                             Collection<LibraryElement> libraries) { }
 
   void assembleProgram() {
     // Conservatively traverse all platform libraries and collect member names.
     // TODO(antonm): ideally we should only collect names of used members,
     // however as of today there are problems with names of some core library
     // interfaces, most probably for interfaces of literals.
     final fixedMemberNames = new Set<String>();
     for (final library in compiler.libraries.getValues()) {
       if (!library.isPlatformLibrary) continue;
-      for (final element in library.localMembers) {
+      // TODO(johnniwinther): Should we use the implementation element to
+      // include injected members?

[ahe, 2012/10/02 13:27:04]

Yes

[Johnni Winther, 2012/10/03 09:22:59]

Done.

+      library.forEachLocalMember((Element element) {
         if (element is ClassElement) {
           ClassElement classElement = element;
           for (final member in classElement.localMembers) {
             final name = member.name.slowToString();
             // Skip operator names.
             if (name.startsWith(@'operator$')) continue;
             // Fetch name of named constructors and factories if any,
             // otherwise store regular name.
             // TODO(antonm): better way to analyze the name.
             fixedMemberNames.add(name.split(@'$').last());
           }
         } else {
           fixedMemberNames.add(element.name.slowToString());
         }
-      }
+      });
     }
     // TODO(antonm): TypeError.srcType and TypeError.dstType are defined in
     // runtime/lib/error.dart. Overall, all DartVM specific libs should be
     // accounted for.
     fixedMemberNames.add('srcType');
     fixedMemberNames.add('dstType');
 
     /**
      * Tells whether we should output given element. Corelib classes like
      * Object should not be in the resulting code.
@@ skipping 204 matching lines @@
 }
 
 compareElements(e0, e1) {
   int result = compareBy((e) => e.getLibrary().uri.toString())(e0, e1);
   if (result != 0) return result;
   return compareBy((e) => e.position().charOffset)(e0, e1);
 }
 
 List<Element> sortElements(Collection<Element> elements) =>
     sorted(elements, compareElements);