Remove parserInsertBefore.

sky/engine/core/html/parser/HTMLConstructionSite.cpp

 /*
  * Copyright (C) 2010 Google, Inc. All Rights Reserved.
  * Copyright (C) 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 57 matching lines @@
 }
 
 static inline void insert(HTMLConstructionSiteTask& task)
 {
     if (isHTMLTemplateElement(*task.parent))
         task.parent = toHTMLTemplateElement(task.parent.get())->content();
 
     if (ContainerNode* parent = task.child->parentNode())
         parent->parserRemoveChild(*task.child);
 
-    if (task.nextChild)
-        task.parent->parserInsertBefore(task.child.get(), *task.nextChild);
-    else
-        task.parent->parserAppendChild(task.child.get());
+    task.parent->parserAppendChild(task.child.get());
 }
 
 static inline void executeInsertTask(HTMLConstructionSiteTask& task)
 {
     ASSERT(task.operation == HTMLConstructionSiteTask::Insert);
 
     insert(task);
 
     if (task.child->isElementNode()) {
         Element& child = toElement(*task.child);
         if (task.selfClosing)
             child.finishParsingChildren();
     }
 }
 
 static inline void executeInsertTextTask(HTMLConstructionSiteTask& task)
 {
     ASSERT(task.operation == HTMLConstructionSiteTask::InsertText);
     ASSERT(task.child->isTextNode());
 
     // Merge text nodes into previous ones if possible:
     // http://www.whatwg.org/specs/web-apps/current-work/multipage/tree-construction.html#insert-a-character
     Text* newText = toText(task.child.get());
-    Node* previousChild = task.nextChild ? task.nextChild->previousSibling() : task.parent->lastChild();
+    Node* previousChild = task.parent->lastChild();
     if (previousChild && previousChild->isTextNode()) {
         Text* previousText = toText(previousChild);
         unsigned lengthLimit = textLengthLimitForContainer(*task.parent);
         if (previousText->length() + newText->length() < lengthLimit) {
             previousText->parserAppendData(newText->data());
             return;
         }
     }
 
     insert(task);
@@ skipping 65 matching lines @@
     const StringBuilder& string = pendingText.stringBuilder;
     while (currentPosition < string.length()) {
         unsigned proposedBreakIndex = std::min(currentPosition + lengthLimit, string.length());
         unsigned breakIndex = findBreakIndexBetween(string, currentPosition, proposedBreakIndex);
         ASSERT(breakIndex <= string.length());
         String substring = string.substring(currentPosition, breakIndex - currentPosition);
         substring = atomizeIfAllWhitespace(substring, pendingText.whitespaceMode);
 
         HTMLConstructionSiteTask task(HTMLConstructionSiteTask::InsertText);
         task.parent = pendingText.parent;
-        task.nextChild = pendingText.nextChild;
         task.child = Text::create(task.parent->document(), substring);
         queueTask(task);
 
         ASSERT(breakIndex > currentPosition);
         ASSERT(breakIndex - currentPosition == substring.length());
         ASSERT(toText(task.child.get())->length() == substring.length());
         currentPosition = breakIndex;
     }
 }
 
@@ skipping 112 matching lines @@
 
 void HTMLConstructionSite::insertTextNode(const String& string, WhitespaceMode whitespaceMode)
 {
     HTMLConstructionSiteTask dummyTask(HTMLConstructionSiteTask::Insert);
     dummyTask.parent = currentNode();
 
     // FIXME: This probably doesn't need to be done both here and in insert(Task).
     if (isHTMLTemplateElement(*dummyTask.parent))
         dummyTask.parent = toHTMLTemplateElement(dummyTask.parent.get())->content();
 
-    // Unclear when parent != case occurs. Somehow we insert text into two separate nodes while processing the same Token.
-    // The nextChild != dummy.nextChild case occurs whenever foster parenting happened and we hit a new text node "<table>a</table>b"
-    // In either case we have to flush the pending text into the task queue before making more.
-    if (!m_pendingText.isEmpty() && (m_pendingText.parent != dummyTask.parent ||  m_pendingText.nextChild != dummyTask.nextChild))
+    // Unclear when parent != case occurs. Somehow we insert text into two separate
+    // nodes while processing the same Token. When it happens we have to flush the
+    // pending text into the task queue before making more.
+    if (!m_pendingText.isEmpty() && (m_pendingText.parent != dummyTask.parent))
         flushPendingText();
-    m_pendingText.append(dummyTask.parent, dummyTask.nextChild, string, whitespaceMode);
+    m_pendingText.append(dummyTask.parent, string, whitespaceMode);
 }
 
 PassRefPtr<Element> HTMLConstructionSite::createElement(AtomicHTMLToken* token, const AtomicString& namespaceURI)
 {
     QualifiedName tagName(token->name());
     RefPtr<Element> element = ownerDocumentForCurrentNode().createElement(tagName, true);
     setAttributes(element.get(), token);
     return element.release();
 }
 
 inline Document& HTMLConstructionSite::ownerDocumentForCurrentNode()
 {
     if (isHTMLTemplateElement(*currentNode()))
         return toHTMLTemplateElement(currentElement())->content()->document();
     return currentNode()->document();
 }
 
 PassRefPtr<HTMLElement> HTMLConstructionSite::createHTMLElement(AtomicHTMLToken* token)
 {
     Document& document = ownerDocumentForCurrentNode();
     RefPtr<HTMLElement> element = HTMLElementFactory::createHTMLElement(token->name(), document, true);
     setAttributes(element.get(), token);
     return element.release();
 }
 
 }