Remove Collection, Collections and clean up List/Set/Queue implementations of retain/remove.

sdk/lib/_internal/compiler/implementation/js_backend/backend.dart

Index: sdk/lib/_internal/compiler/implementation/js_backend/backend.dart
diff --git a/sdk/lib/_internal/compiler/implementation/js_backend/backend.dart b/sdk/lib/_internal/compiler/implementation/js_backend/backend.dart
index f3ad005bb4a3d4bf89dbb780f0606550161dd401..388ba4313c31229951d6479aec5c1cb8d62aad9b 100644
--- a/sdk/lib/_internal/compiler/implementation/js_backend/backend.dart
+++ b/sdk/lib/_internal/compiler/implementation/js_backend/backend.dart
@@ -734,7 +734,7 @@ class JavaScriptBackend extends Backend {
    * the generic version that contains all possible type checks is
    * also stored in this map.
    */
-  final Map<String, Collection<ClassElement>> specializedGetInterceptors;
+  final Map<String, Iterable<ClassElement>> specializedGetInterceptors;

[floitsch, 2013/04/11 15:17:50]

I think this should be a Set.

[Anders Johnsen, 2013/04/12 09:31:14]

Done.

 
   /**
    * Set of classes whose methods are intercepted.
@@ -768,7 +768,7 @@ class JavaScriptBackend extends Backend {
         interceptedElements = new Map<SourceString, Set<Element>>(),
         rti = new RuntimeTypes(compiler),
         specializedGetInterceptors =
-            new Map<String, Collection<ClassElement>>(),
+            new Map<String, Iterable<ClassElement>>(),

[floitsch, 2013/04/11 15:17:50]

Set

[Anders Johnsen, 2013/04/12 09:31:14]

Done.

         super(compiler, JAVA_SCRIPT_CONSTANT_SYSTEM) {
     emitter = disableEval
         ? new CodeEmitterNoEvalTask(compiler, namer, generateSourceMap)