Change MirrorSystem.libraries to Map<Uri, String>

sdk/lib/_internal/compiler/implementation/mirrors/mirrors.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.
 
 library mirrors;
 
 import 'dart:async';
 import 'dart:uri';
 
 // TODO(rnystrom): Use "package:" URL (#4968).
 import 'dart2js_mirror.dart';
 
 /**
  * The main interface for the whole mirror system.
  */
 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.
    */
   TypeMirror get dynamicType;
 
   /**
    * A mirror on the [:void:] type.
    */
   TypeMirror get voidType;
@@ skipping 659 matching lines @@
   /**
    * Returns the URI where the source originated.
    */
   Uri get sourceUri;
 
   /**
    * Returns the text of this source.
    */
   String get sourceText;
 }