Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(231)

Unified Diff: sdk/lib/html/dartium/html_dartium.dart

Issue 12596004: Replaced Element.append with Node.appendChild (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« no previous file with comments | « sdk/lib/html/dart2js/html_dart2js.dart ('k') | tests/html/node_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk/lib/html/dartium/html_dartium.dart
diff --git a/sdk/lib/html/dartium/html_dartium.dart b/sdk/lib/html/dartium/html_dartium.dart
index e1219575b74eb244c8a0528ac508bc9c663c7e6c..1a5fa2d5b2ea4feb17254c1424573d15dd3c143a 100644
--- a/sdk/lib/html/dartium/html_dartium.dart
+++ b/sdk/lib/html/dartium/html_dartium.dart
@@ -7655,7 +7655,7 @@ class DocumentFragment extends Node {
String get innerHtml {
final e = new Element.tag("div");
- e.nodes.add(this.clone(true));
+ e.append(this.clone(true));
return e.innerHtml;
}
@@ -7673,19 +7673,11 @@ class DocumentFragment extends Node {
}
/**
- * Adds the specified element after the last child of this
- * document fragment.
- */
- void append(Element element) {
- this.children.add(element);
- }
-
- /**
* Adds the specified text as a text node after the last child of this
* document fragment.
*/
void appendText(String text) {
- this.nodes.add(new Text(text));
+ this.append(new Text(text));
}
@@ -7694,7 +7686,7 @@ class DocumentFragment extends Node {
* last child of this document fragment.
*/
void appendHtml(String text) {
- this.nodes.add(new DocumentFragment.html(text));
+ this.append(new DocumentFragment.html(text));
}
DocumentFragment.internal() : super.internal();
@@ -8910,7 +8902,7 @@ class _ChildrenElementList implements List {
}
Element add(Element value) {
- _element.$dom_appendChild(value);
+ _element.append(value);
return value;
}
@@ -8924,7 +8916,7 @@ class _ChildrenElementList implements List {
}
for (Element element in iterable) {
- _element.$dom_appendChild(element);
+ _element.append(element);
}
}
@@ -9542,13 +9534,6 @@ abstract class Element extends Node implements ElementTraversal {
$dom_offsetHeight);
/**
- * Adds the specified element to after the last child of this element.
- */
- void append(Element e) {
- this.children.add(e);
- }
-
- /**
* Adds the specified text as a text node after the last child of this
* element.
*/
@@ -18888,11 +18873,11 @@ class _ChildNodeListLazy implements List {
}
void add(Node value) {
- _this.$dom_appendChild(value);
+ _this.append(value);
}
void addLast(Node value) {
- _this.$dom_appendChild(value);
+ _this.append(value);
}
@@ -18902,13 +18887,13 @@ class _ChildNodeListLazy implements List {
// Optimized route for copying between nodes.
for (var i = 0, len = iterable.length; i < len; ++i) {
// Should use $dom_firstChild, Bug 8886.
- _this.$dom_appendChild(iterable[0]);
+ _this.append(iterable[0]);
}
}
return;
}
for (Node node in iterable) {
- _this.$dom_appendChild(node);
+ _this.append(node);
}
}
@@ -19092,7 +19077,7 @@ class Node extends EventTarget {
List copy = new List.from(value);
text = '';
for (Node node in copy) {
- $dom_appendChild(node);
+ append(node);
}
}
@@ -19212,7 +19197,7 @@ class Node extends EventTarget {
@DomName('Node.appendChild')
@DocsEditable
- Node $dom_appendChild(Node newChild) native "Node_appendChild_Callback";
+ Node append(Node newChild) native "Node_appendChild_Callback";
@DomName('Node.cloneNode')
@DocsEditable
« no previous file with comments | « sdk/lib/html/dart2js/html_dart2js.dart ('k') | tests/html/node_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698