fix(dccd): fix removals reporting

closes dart-archive#1097

Author: vicb

lib/change_detection/dirty_checking_change_detector.dart

@@ -1255,6 +1255,7 @@ class _CollectionChangeRecord<V> implements CollectionChangeRecord<V> {
   ItemRecord<V> _addToRemovals(ItemRecord<V> record) {
     _removedItems.put(record);
     record.currentIndex = null;
+    record._nextRemoved = null;
 
     if (_removalsTail == null) {
       assert(_removalsHead == null);

test/change_detection/dirty_checking_change_detector_spec.dart

@@ -288,6 +288,31 @@ void testWithGetterFactory(FieldGetterFactory getterFactory) {
               moves: ['2[1 -> 0]', '1[0 -> 1]'],
               removals: []));
         });
+
+        it('should handle swapping elements correctly - gh1097', () {
+          // This test would only have failed in non-checked mode only
+          var list = ['a', 'b', 'c'];
+          var record = detector.watch(list, null, null);
+          var iterator = detector.collectChanges()..moveNext();
+
+          list..clear()..addAll(['b', 'a', 'c']);
+          iterator = detector.collectChanges()..moveNext();
+          expect(iterator.current.currentValue, toEqualCollectionRecord(
+              collection: ['b[1 -> 0]', 'a[0 -> 1]', 'c'],
+              previous: ['a[0 -> 1]', 'b[1 -> 0]', 'c'],
+              additions: [],
+              moves: ['b[1 -> 0]', 'a[0 -> 1]', 'c'],
+              removals: []));
+
+          list..clear()..addAll(['b', 'c', 'a']);
+          iterator = detector.collectChanges()..moveNext();
+          expect(iterator.current.currentValue, toEqualCollectionRecord(
+              collection: ['b', 'c[2 -> 1]', 'a[1 -> 2]'],
+              previous: ['b', 'a[1 -> 2]', 'c[2 -> 1]'],
+              additions: [],
+              moves: ['c[2 -> 1]', 'a[1 -> 2]'],
+              removals: []));
+        });
       });
 
       it('should detect changes in list', () {