Change MirrorSystem.libraries to Map<Uri, String>

sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart

Index: sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart
diff --git a/sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart b/sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart
index cec6c29b3ab8940e0d4393f581fadb2cd69c5231..cd435bc3d5f1905479f176587e6b18efc9a15dc1 100644
--- a/sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart
+++ b/sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart
@@ -17,8 +17,7 @@ abstract class MirrorSystem {
   /**
    * Returns an unmodifiable map of all libraries in this mirror system.
    */
-  // TODO(johnniwinther): Change to Map<Uri, LibraryMirror>.
-  Map<String, LibraryMirror> get libraries;
+  Map<Uri, LibraryMirror> get libraries;

[gbracha, 2013/04/08 20:29:10]

When was this API change discussed? As far as I can tell, URIs are not
canonicalized. And so, the question is how does a URI is reliably identify a
library?  In other words, how do I find a given library in this map?

[Johnni Winther, 2013/04/22 13:00:02]

It wasn't prior. I made this CL to get your input.

 
   /**
    * A mirror on the [:dynamic:] type.