Fix _ChildrenNodeList#forEach.

client/html/src/NodeWrappingImplementation.dart

Index: client/html/src/NodeWrappingImplementation.dart
diff --git a/client/html/src/NodeWrappingImplementation.dart b/client/html/src/NodeWrappingImplementation.dart
index e44f063d90561367ea38532a67d75f848a251d21..f677df58939e1513a52d593eebd84b90980f668f 100644
--- a/client/html/src/NodeWrappingImplementation.dart
+++ b/client/html/src/NodeWrappingImplementation.dart
@@ -24,8 +24,8 @@ class _ChildrenNodeList implements NodeList {
   }
 
   void forEach(void f(Node element)) {
-    for (var node in _childNodes) {
-      f(LevelDom.wrapNode(node));
+    for (int i = 0, len = _childNodes.length; i < len; i++) {

[Jacob, 2011/12/14 23:00:13]

what if the children change while iterating?
to be safe we have to copy the children to a temp array
or mitigate the issue somewhat by not caching len and adding a comment warning
of the edge case.
 

[nweiz, 2011/12/14 23:21:38]

That makes me realize, this could be implemented much more cleanly by just using
_toList.

+      f(LevelDom.wrapNode(_childNodes[i]));
     }
   }