Use Atree non-readonly iterators in Transfer()

If remove parameter is true in Transfer(), iterated values are
removed so non-readonly iterators need to be used.

runtime/interpreter/value.go

@@ -2738,7 +2738,9 @@ func (v *ArrayValue) Transfer(
 
 	if needsStoreTo || !isResourceKinded {
 
-		iterator, err := v.array.ReadOnlyIterator()
+		// Use non-readonly iterator here because iterated
+		// value can be removed if remove parameter is true.
+		iterator, err := v.array.Iterator()
 		if err != nil {
 			panic(errors.NewExternalError(err))
 		}
@@ -17302,7 +17304,12 @@ func (v *CompositeValue) Transfer(
 	}
 
 	if needsStoreTo || !isResourceKinded {
-		iterator, err := v.dictionary.ReadOnlyIterator()
+		// Use non-readonly iterator here because iterated
+		// value can be removed if remove parameter is true.
+		iterator, err := v.dictionary.Iterator(
+			StringAtreeValueComparator,
+			StringAtreeValueHashInput,
+		)
 		if err != nil {
 			panic(errors.NewExternalError(err))
 		}
@@ -18918,7 +18925,9 @@ func (v *DictionaryValue) Transfer(
 		valueComparator := newValueComparator(interpreter, locationRange)
 		hashInputProvider := newHashInputProvider(interpreter, locationRange)
 
-		iterator, err := v.dictionary.ReadOnlyIterator()
+		// Use non-readonly iterator here because iterated
+		// value can be removed if remove parameter is true.
+		iterator, err := v.dictionary.Iterator(valueComparator, hashInputProvider)
 		if err != nil {
 			panic(errors.NewExternalError(err))
 		}