Don't use ReversedListView from outside core and collection.

samples/swarm/swarm_ui_lib/observable/observable.dart

 // Copyright (c) 2011, 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 observable;
 
 import 'dart:collection';
 import 'dart:collection-dev';

[Lasse Reichstein Nielsen, 2013/01/29 08:11:22]

But collections-dev is imported here, so why shouldn't it work?

[floitsch, 2013/01/29 15:41:07]

It shouldn't. I will prepare to remove it.

 
 part 'ChangeEvent.dart';
 part 'EventBatch.dart';
 
 /**
  * An object whose changes are tracked and who can issue events notifying how it
  * has been changed.
  */
 abstract class Observable {
   /** Returns a globally unique identifier for the object. */
@@ skipping 137 matching lines @@
   void set length(int value) {
     _internal.length = value;
     recordGlobalChange();
   }
 
   void clear() {
     _internal.clear();
     recordGlobalChange();
   }
 
-  List<T> get reversed => new ReversedListView<T>(this, 0, null);
+  List<T> get reversed => _internal.reversed;

[Lasse Reichstein Nielsen, 2013/01/29 08:11:22]

That works, but only because you can't assign through the reversed list.

[floitsch, 2013/01/29 15:41:07]

Should be ok then. If we change that behavior, we shouldn't forget to change it
here. Even if we forget, it should not be a big problem, since Swarm doesn't use
"reversed".

 
   void sort([int compare(var a, var b)]) {
     if (compare == null) compare = Comparable.compare;
     _internal.sort(compare);
     recordGlobalChange();
   }
 
   void add(T element) {
     recordListInsert(length, element);
     _internal.add(element);
@@ skipping 101 matching lines @@
     throw new UnimplementedError();
   }
 
 
   // Iterable<T>:
   Iterator<T> get iterator => _internal.iterator;
 
   // Collection<T>:
   Iterable<T> where(bool f(T element)) => _internal.where(f);
   Iterable mappedBy(f(T element)) => _internal.mappedBy(f);
+  List<T> skip(int count) => _internal.skip(count);
+  List<T> take(int count) => _internal.take(count);
   bool every(bool f(T element)) => _internal.every(f);
   bool any(bool f(T element)) => _internal.any(f);
   void forEach(void f(T element)) { _internal.forEach(f); }
   String join([String separator]) => _internal.join(separator);
   T firstMatching(bool test(T value), {T orElse()}) {
     return _internal.firstMatching(test, orElse: orElse);
   }
   T lastMatching(bool test(T value), {T orElse()}) {
     return _internal.lastMatching(test, orElse: orElse);
   }
@@ skipping 24 matching lines @@
     // Only fire on an actual change.
     if (!identical(newValue, _value)) {
       final oldValue = _value;
       _value = newValue;
       recordPropertyUpdate("value", newValue, oldValue);
     }
   }
 
   T _value;
 }