Refactor Source and SourceRegistry.

lib/src/source_registry.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.
 
-import 'dart:collection';
-
 import 'package.dart';
 import 'source.dart';
+import 'source/git.dart';
+import 'source/hosted.dart';
+import 'source/path.dart';
 import 'source/unknown.dart';
 
 /// A class that keeps track of [Source]s used for getting packages.
-class SourceRegistry extends IterableBase<Source> {
-  final _sources = new Map<String, Source>();
+class SourceRegistry {
+  /// The registered sources.
+  ///
+  /// This is initialized with the three built-in sources.
+  final _sources = {
+    "git": new GitSource(),
+    "hosted": new HostedSource(),
+    "path": new PathSource()
+  };
+
+  /// The default source, which is used when no source is specified.
+  ///
+  /// This defaults to [hosted].
+  Source get defaultSource => _default;
   Source _default;
 
-  /// Returns the default source, which is used when no source is specified.
-  Source get defaultSource => _default;
-
-  /// Iterates over the registered sources in name order.
-  Iterator<Source> get iterator {
+  /// The registered sources, in name order.
+  List<Source> get sources {

[Bob Nystrom, 2016/06/14 23:21:55]

What do you think of calling this "all"?

Elsewhere, we tend to store a reference to the registry in variable named
"sources", and "sources.sources" looks weird.

[nweiz, 2016/06/20 20:46:08]

Done.

     var sources = _sources.values.toList();
     sources.sort((a, b) => a.name.compareTo(b.name));
-    return sources.iterator;
+    return sources;
+  }
+
+  /// The built-in [GitSource].
+  GitSource get git => _sources["git"] as GitSource;
+
+  /// The built-in [HostedSource].
+  HostedSource get hosted => _sources["hosted"] as HostedSource;
+
+  /// The built-in [PathSource].
+  PathSource get path => _sources["path"] as PathSource;
+
+  SourceRegistry() {
+    _default = hosted;
   }
 
   /// Returns whether [id1] and [id2] refer to the same package, including
   /// validating that their descriptions are equivalent.
   bool idsEqual(PackageId id1, PackageId id2) {
     if (id1 != id2) return false;
     if (id1 == null && id2 == null) return true;
     return idDescriptionsEqual(id1, id2);
   }
 
@@ skipping 22 matching lines @@
   /// registered.
   void register(Source source) {
     if (_sources.containsKey(source.name)) {
       throw new StateError('Source registry already has a source named '
           '${source.name}');
     }
 
     _sources[source.name] = source;
   }
 
-  /// Loads the package identified by [id].
-  ///
-  /// Throws an [ArgumentError] if [id] has an invalid source.
-  Package load(PackageId id) {
-    var source = this[id.source];
-    if (source == null) throw new ArgumentError("Unknown source ${id.source}.");
-
-    var dir = source.getDirectory(id);
-    return new Package.load(id.name, dir, this);
-  }
-
   /// Returns the source named [name].
   ///
   /// Returns an [UnknownSource] if no source with that name has been
   /// registered. If [name] is null, returns the default source.
   Source operator[](String name) {
-    if (name == null) {
-      if (defaultSource != null) return defaultSource;
-      throw new StateError('No default source has been registered');
-    }
-
+    if (name == null) return _default;
     if (_sources.containsKey(name)) return _sources[name];
     return new UnknownSource(name);
   }
 }