Add and use AnalysisCache.sources getter.

pkg/analyzer/lib/src/context/context.dart

 // Copyright (c) 2015, 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 analyzer.src.context.context;
 
 import 'dart:async';
 import 'dart:collection';
 
 import 'package:analyzer/src/cancelable_future.dart';
@@ skipping 273 matching lines @@
   }
 
   @override
   List<Source> get htmlSources => _getSources(SourceKind.HTML);
 
   @override
   bool get isDisposed => _disposed;
 
   @override
   List<Source> get launchableClientLibrarySources {
-    List<Source> sources = new List<Source>();
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      AnalysisTarget target = iterator.key;
-      CacheEntry entry = iterator.value;
-      if (target is Source &&
-          entry.getValue(SOURCE_KIND) == SourceKind.LIBRARY &&
-          !target.isInSystemLibrary &&
-          isClientLibrary(target)) {
-        sources.add(target);
+    List<Source> sources = <Source>[];
+    for (Source source in _cache.sources) {
+      CacheEntry entry = _cache.get(source);
+      if (entry.getValue(SOURCE_KIND) == SourceKind.LIBRARY &&
+          !source.isInSystemLibrary &&
+          isClientLibrary(source)) {
+        sources.add(source);
       }
     }
     return sources;
   }
 
   @override
   List<Source> get launchableServerLibrarySources {
-    List<Source> sources = new List<Source>();
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      AnalysisTarget target = iterator.key;
-      CacheEntry entry = iterator.value;
-      if (target is Source &&
+    List<Source> sources = <Source>[];
+    for (Source source in _cache.sources) {
+      CacheEntry entry = _cache.get(source);
+      if (source is Source &&
           entry.getValue(SOURCE_KIND) == SourceKind.LIBRARY &&
-          !target.isInSystemLibrary &&
-          isServerLibrary(target)) {
-        sources.add(target);
+          !source.isInSystemLibrary &&
+          isServerLibrary(source)) {
+        sources.add(source);
       }
     }
     return sources;
   }
 
   @override
   List<Source> get librarySources => _getSources(SourceKind.LIBRARY);
 
   @override
   Stream<SourcesChangedEvent> get onSourcesChanged =>
@@ skipping 26 matching lines @@
       _sourceFactory.context = null;
     }
     factory.context = this;
     _sourceFactory = factory;
     _cache = createCacheFromSourceFactory(factory);
     _invalidateAllLocalResolutionInformation(true);
   }
 
   @override
   List<Source> get sources {
-    List<Source> sources = new List<Source>();
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      AnalysisTarget target = iterator.key;
-      if (target is Source) {
-        sources.add(target);
-      }
-    }
-    return sources;
+    return _cache.sources.toList();
   }
 
   /**
    * Return a list of the sources that would be processed by
    * [performAnalysisTask]. This method duplicates, and must therefore be kept
    * in sync with, [getNextAnalysisTask]. This method is intended to be used for
    * testing purposes only.
    */
   List<Source> get sourcesNeedingProcessing {
     HashSet<Source> sources = new HashSet<Source>();
@@ skipping 376 matching lines @@
 //    }
     return null;
   }
 
   @override
   List<Source> getHtmlFilesReferencing(Source source) {
     SourceKind sourceKind = getKindOf(source);
     if (sourceKind == null) {
       return Source.EMPTY_LIST;
     }
-    List<Source> htmlSources = new List<Source>();
+    List<Source> htmlSources = <Source>[];
     while (true) {
       if (sourceKind == SourceKind.PART) {
         List<Source> librarySources = getLibrariesContaining(source);
-        MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-        while (iterator.moveNext()) {
-          CacheEntry entry = iterator.value;
+        for (Source source in _cache.sources) {
+          CacheEntry entry = _cache.get(source);
           if (entry.getValue(SOURCE_KIND) == SourceKind.HTML) {
             List<Source> referencedLibraries =
                 (entry as HtmlEntry).getValue(HtmlEntry.REFERENCED_LIBRARIES);
             if (_containsAny(referencedLibraries, librarySources)) {
-              htmlSources.add(iterator.key);
+              htmlSources.add(source);
             }
           }
         }
       } else {
-        MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-        while (iterator.moveNext()) {
-          CacheEntry entry = iterator.value;
+        for (Source source in _cache.sources) {
+          CacheEntry entry = _cache.get(source);
           if (entry.getValue(SOURCE_KIND) == SourceKind.HTML) {
             List<Source> referencedLibraries =
                 (entry as HtmlEntry).getValue(HtmlEntry.REFERENCED_LIBRARIES);
             if (_contains(referencedLibraries, source)) {
-              htmlSources.add(iterator.key);
+              htmlSources.add(source);
             }
           }
         }
       }
       break;
     }
     if (htmlSources.isEmpty) {
       return Source.EMPTY_LIST;
     }
     return htmlSources;
@@ skipping 10 matching lines @@
     return SourceKind.UNKNOWN;
   }
 
   @override
   List<Source> getLibrariesContaining(Source source) {
     SourceKind kind = getKindOf(source);
     if (kind == SourceKind.LIBRARY) {
       return <Source>[source];
     } else if (kind == SourceKind.PART) {
       List<Source> libraries = <Source>[];
-      MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-      while (iterator.moveNext()) {
-        AnalysisTarget target = iterator.key;
-        if (target is Source && getKindOf(target) == SourceKind.LIBRARY) {
-          List<Source> parts = _cache.getValue(target, INCLUDED_PARTS);
+      for (Source library in _cache.sources) {
+        CacheEntry entry = _cache.get(library);
+        if (entry.getValue(SOURCE_KIND) == SourceKind.LIBRARY) {
+          List<Source> parts = entry.getValue(INCLUDED_PARTS);
           if (parts.contains(source)) {
-            libraries.add(target);
+            libraries.add(library);
           }
         }
       }
       if (libraries.isNotEmpty) {
         return libraries;
       }
     }
     return Source.EMPTY_ARRAY;
   }
 
   @override
   List<Source> getLibrariesDependingOn(Source librarySource) {
-    List<Source> dependentLibraries = new List<Source>();
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      CacheEntry entry = iterator.value;
+    List<Source> dependentLibraries = <Source>[];
+    for (Source source in _cache.sources) {
+      CacheEntry entry = _cache.get(source);
       if (entry.getValue(SOURCE_KIND) == SourceKind.LIBRARY) {
         if (_contains(entry.getValue(EXPORTED_LIBRARIES), librarySource)) {
-          dependentLibraries.add(iterator.key);
+          dependentLibraries.add(source);
         }
         if (_contains(entry.getValue(IMPORTED_LIBRARIES), librarySource)) {
-          dependentLibraries.add(iterator.key);
+          dependentLibraries.add(source);
         }
       }
     }
     if (dependentLibraries.isEmpty) {
       return Source.EMPTY_LIST;
     }
     return dependentLibraries;
   }
 
   @override
@@ skipping 81 matching lines @@
 //    if (sourceEntry is HtmlEntry) {
 //      HtmlEntry htmlEntry = sourceEntry;
 //      return htmlEntry.getValue(HtmlEntry.RESOLVED_UNIT);
 //    }
     return null;
   }
 
   @override
   List<Source> getSourcesWithFullName(String path) {
     List<Source> sources = <Source>[];
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      AnalysisTarget target = iterator.key;
-      if (target is Source && target.fullName == path) {
-        sources.add(target);
+    for (Source source in _cache.sources) {
+      if (source.fullName == path) {
+        sources.add(source);
       }
     }
     return sources;
   }
 
   @override
   bool handleContentsChanged(
       Source source, String originalContents, String newContents, bool notify) {
     CacheEntry entry = _cache.get(source);
     if (entry == null) {
@@ skipping 272 matching lines @@
    */
   void visitContentCache(ContentCacheVisitor visitor) {
     _contentCache.accept(visitor);
   }
 
   /**
    * Add all of the sources contained in the given source [container] to the
    * given list of [sources].
    */
   void _addSourcesInContainer(List<Source> sources, SourceContainer container) {
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      Source source = iterator.key;
+    for (Source source in _cache.sources) {
       if (container.contains(source)) {
         sources.add(source);
       }
     }
   }
 
   /**
    * Remove the given [pendingFuture] from [_pendingFutureTargets], since the
    * client has indicated its computation is not needed anymore.
    */
@@ skipping 147 matching lines @@
     List<ChangeNotice> notices = new List.from(_pendingNotices.values);
     _pendingNotices.clear();
     return notices;
   }
 
   /**
    * Return a list containing all of the sources known to this context that have
    * the given [kind].
    */
   List<Source> _getSources(SourceKind kind) {
-    List<Source> sources = new List<Source>();
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      if (iterator.value.getValue(SOURCE_KIND) == kind &&
-          iterator.key is Source) {
-        sources.add(iterator.key);
+    List<Source> sources = <Source>[];
+    for (Source source in _cache.sources) {
+      CacheEntry entry = _cache.get(source);
+      if (entry.getValue(SOURCE_KIND) == kind) {
+        sources.add(source);
       }
     }
     return sources;
   }
 
   /**
    * Look at the given [source] to see whether a task needs to be performed
    * related to it. If so, add the source to the set of sources that need to be
    * processed. This method is intended to be used for testing purposes only.
    */
@@ skipping 178 matching lines @@
 //        }
 //      }
 //    });
   }
 
   /**
    * Remove the given [source] from the priority order if it is in the list.
    */
   void _removeFromPriorityOrder(Source source) {
     int count = _priorityOrder.length;
-    List<Source> newOrder = new List<Source>();
+    List<Source> newOrder = <Source>[];
     for (int i = 0; i < count; i++) {
       if (_priorityOrder[i] != source) {
         newOrder.add(_priorityOrder[i]);
       }
     }
     if (newOrder.length < count) {
       analysisPriorityOrder = newOrder;
     }
   }
 
@@ skipping 163 matching lines @@
   /**
    * Check the cache for any invalid entries (entries whose modification time
    * does not match the modification time of the source associated with the
    * entry). Invalid entries will be marked as invalid so that the source will
    * be re-analyzed. Return `true` if at least one entry was invalid.
    */
   bool _validateCacheConsistency() {
     int consistencyCheckStart = JavaSystem.nanoTime();
     HashSet<Source> changedSources = new HashSet<Source>();
     HashSet<Source> missingSources = new HashSet<Source>();
-    MapIterator<AnalysisTarget, CacheEntry> iterator = _cache.iterator();
-    while (iterator.moveNext()) {
-      AnalysisTarget target = iterator.key;
-      if (target is Source) {
-        CacheEntry entry = iterator.value;
-        int sourceTime = getModificationStamp(target);
-        if (sourceTime != entry.modificationTime) {
-          changedSources.add(target);
-        }
-        if (entry.exception != null) {
-          if (!exists(target)) {
-            missingSources.add(target);
-          }
+    for (Source source in _cache.sources) {
+      CacheEntry entry = _cache.get(source);
+      int sourceTime = getModificationStamp(source);
+      if (sourceTime != entry.modificationTime) {
+        changedSources.add(source);
+      }
+      if (entry.exception != null) {
+        if (!exists(source)) {
+          missingSources.add(source);
         }
       }
     }
     for (Source source in changedSources) {
       _sourceChanged(source);
     }
     int removalCount = 0;
     for (Source source in missingSources) {
       if (getLibrariesContaining(source).isEmpty &&
           getLibrariesDependingOn(source).isEmpty) {
@@ skipping 204 matching lines @@
     PendingFuture pendingFuture =
         new PendingFuture<T>(_context, target, computeValue);
     if (!pendingFuture.evaluate(entry)) {
       _context._pendingFutureTargets
           .putIfAbsent(target, () => <PendingFuture>[])
           .add(pendingFuture);
     }
     return pendingFuture.future;
   }
 }