Morph the APIs for Node, ParentNode, and Element closer to the specs

sky/engine/core/editing/RemoveNodePreservingChildrenCommand.cpp

Index: sky/engine/core/editing/RemoveNodePreservingChildrenCommand.cpp
diff --git a/sky/engine/core/editing/RemoveNodePreservingChildrenCommand.cpp b/sky/engine/core/editing/RemoveNodePreservingChildrenCommand.cpp
index 9f991dbc853be6ea6d9b43ec3af37001e7e9ad53..b375646ed466940c3fcf271b8f20956991e13a00 100644
--- a/sky/engine/core/editing/RemoveNodePreservingChildrenCommand.cpp
+++ b/sky/engine/core/editing/RemoveNodePreservingChildrenCommand.cpp
@@ -43,7 +43,7 @@ void RemoveNodePreservingChildrenCommand::doApply()
 {
     if (m_node->isContainerNode()) {
         NodeVector children;
-        getChildNodes(toContainerNode(*m_node), children);
+        appendChildNodes(toContainerNode(*m_node), children);

[eseidel, 2015/02/14 00:12:23]

I think this is going to be very confusing to read.  I think we're going to need
some sort of takeNodesAndAppend or similar.

[abarth-chromium, 2015/02/14 00:27:19]

I can change this back to getChildNodes.  The problem was that there's now a
getChildNodes function in Node that's part of the API.  This does almost the
same thing, except NodeVector has some inline capacity.

 
         size_t size = children.size();
         for (size_t i = 0; i < size; ++i) {