Chromium Code Reviews| Index: client/dom/templates/html/impl/impl_Node.darttemplate |
| diff --git a/client/dom/templates/html/impl/impl_Node.darttemplate b/client/dom/templates/html/impl/impl_Node.darttemplate |
| index 893b877649bd1e55eb4bda52f196e6d76d9b03dc..531b64bc24460e75cd8dded64a89663457950408 100644 |
| --- a/client/dom/templates/html/impl/impl_Node.darttemplate |
| +++ b/client/dom/templates/html/impl/impl_Node.darttemplate |
| @@ -13,8 +13,10 @@ class $CLASSNAME$EXTENDS$IMPLEMENTS$NATIVESPEC { |
| // Copy list first since we don't want liveness during iteration. |
| // TODO(jacobr): there is a better way to do this. |
| List copy = new List.from(value); |
|
vsm
2012/03/05 21:37:36
I don't think this extra copy is buying anything.
Jacob
2012/03/05 21:55:30
This code already has a TODO that it is doing thin
|
| - nodes.clear(); |
| - nodes.addAll(copy); |
| + text = ''; |
|
sra1
2012/03/05 21:44:59
What does this do? Does setting text like this re
Jacob
2012/03/05 21:55:30
that's the goal. remove all the old nodes and add
|
| + for (Node node in copy) { |
| + _appendChild(node); |
| + } |
| } |
| // TODO(jacobr): should we throw an exception if parent is already null? |