Support imports for dart: libs.

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.
  */
 class ConflictingRenamer extends Renamer {
   final Compiler compiler;
   final Map<Element, String> renamed;
   final Set<String> usedTopLevelIdentifiers;
+  final Map<LibraryElement, String> imports;
   TreeElements contextElements;
   Element context;
 
   Map<Element, TreeElements> get resolvedElements() =>
       compiler.enqueuer.resolution.resolvedElements;
 
   ConflictingRenamer(this.compiler) :
       renamed = new Map<Element, String>(),
-      usedTopLevelIdentifiers = new Set<String>();
+      usedTopLevelIdentifiers = new Set<String>(),
+      imports = new Map<LibraryElement, String>();
 
   void setContext(Element element) {
     this.context = element;
     contextElements = resolvedElements[element];
   }
 
   String getFactoryName(FunctionExpression node) =>
       node.name.asSend().selector.asIdentifier().source.slowToString();
 
   bool isNamedConstructor(Element element) =>
@@ skipping 43 matching lines @@
         return renameElement(enclosingClass);
       }
     }
     if (context.isFunction() && context.cachedNode.name == node) {
       return renameElement(context);
     }
     return null;
   }
 
   String renameElement(Element element) {
+    generateUniqueName(name) {
+      while (usedTopLevelIdentifiers.contains(name)) {
+        name = "x$name";

[Roman, 2012/07/30 14:43:30]

Why 'x'? 'x' is an ordinary letter, it will be harder to distinguish prefixed
and original names. I would prefer '_' or some other 'special' character.

[Anton Muhin, 2012/07/30 14:54:31]

Agree, but apparently due to the bug in DartVM, '_p' works poorly as a prefix:
_p.Foo is not resolved :(

I'm going to double check and file a bug.

On 2012/07/30 14:43:30, Roman wrote:

[Roman, 2012/07/30 15:03:42]

That's sad. Maybe '$' ? or anything else?

[Anton Muhin, 2012/07/30 15:37:16]

Done.

+      }
+      usedTopLevelIdentifiers.add(name);
+      return name;
+    }
+
     String originalName = element.name.slowToString();
-    if (element.getLibrary() == compiler.coreLibrary || !element.isTopLevel()) {
-      return originalName;
+    // TODO(antonm): we should rename lib private names as well.
+    if (!element.isTopLevel()) return originalName;
+    final library = element.getLibrary();
+    if (isDartCoreLib(compiler, library)) {
+      final prefix = imports.putIfAbsent(library, () => generateUniqueName('p'));

[Roman, 2012/07/30 14:43:30]

Why 'p' as a prefix? Easier to read would be something like
library.getLibraryOrScriptName()

[Anton Muhin, 2012/07/30 14:54:31]

I am not sure it's a valid Dart identifier.  Plus it would be just too long.  I
picked p almost randomly (p stands for prefix).

[Roman, 2012/07/30 15:03:42]

We can fix it if for some reason this will not be an identifier. And, we're not
writing a minifier here, so no need to create identifiers as small as possible,
but it may be very useful for debugging to see the original prefix.

+      return '$prefix.$originalName';
     }
     if (renamed[element] !== null) {

[Roman, 2012/07/30 14:43:30]

Should this check be moved to before generating renamed dart lib prefix? Just to
not calculate the same thing several times.

[Anton Muhin, 2012/07/30 14:54:31]

Done.

       return renamed[element];
     }
 
     // Not renamed and top element.
     // TODO(smok): Make sure that the new name does not conflict with existing
     // local identifiers.
-    String name = originalName;
-    while (usedTopLevelIdentifiers.contains(name)) {
-      name = "_$name";
-    }
-    usedTopLevelIdentifiers.add(name);
-    renamed[element] = name;
-    return name;
+    return renamed[element] = generateUniqueName(originalName);
   }
 }