Replace WTF::move with WTFMove
[WebKit-https.git] / Source / WebCore / html / HTMLTableRowElement.cpp
index c87259f..766bd0c 100644 (file)
@@ -28,7 +28,7 @@
 #include "ExceptionCode.h"
 #include "GenericCachedHTMLCollection.h"
 #include "HTMLNames.h"
-#include "HTMLTableCellElement.h"
+#include "HTMLTableDataCellElement.h"
 #include "HTMLTableElement.h"
 #include "HTMLTableSectionElement.h"
 #include "NodeList.h"
@@ -57,89 +57,73 @@ Ref<HTMLTableRowElement> HTMLTableRowElement::create(const QualifiedName& tagNam
 
 int HTMLTableRowElement::rowIndex() const
 {
-    ContainerNode* table = parentNode();
-    if (!table)
+    auto* parent = parentNode();
+    if (!parent)
         return -1;
-    table = table->parentNode();
-    if (!is<HTMLTableElement>(table))
-        return -1;
-
-    // To match Firefox, the row indices work like this:
-    //   Rows from the first <thead> are numbered before all <tbody> rows.
-    //   Rows from the first <tfoot> are numbered after all <tbody> rows.
-    //   Rows from other <thead> and <tfoot> elements don't get row indices at all.
-
-    int rIndex = 0;
 
-    if (HTMLTableSectionElement* head = downcast<HTMLTableElement>(*table).tHead()) {
-        for (Node *row = head->firstChild(); row; row = row->nextSibling()) {
-            if (row == this)
-                return rIndex;
-            if (row->hasTagName(trTag))
-                ++rIndex;
-        }
-    }
-    
-    for (Node *node = table->firstChild(); node; node = node->nextSibling()) {
-        if (node->hasTagName(tbodyTag)) {
-            HTMLTableSectionElement& section = downcast<HTMLTableSectionElement>(*node);
-            for (Node* row = section.firstChild(); row; row = row->nextSibling()) {
-                if (row == this)
-                    return rIndex;
-                if (row->hasTagName(trTag))
-                    ++rIndex;
-            }
-        }
+    HTMLTableElement* table;
+    if (is<HTMLTableElement>(*parent))
+        table = downcast<HTMLTableElement>(parent);
+    else {
+        if (!is<HTMLTableSectionElement>(*parent) || !is<HTMLTableElement>(parent->parentNode()))
+            return -1;
+        table = downcast<HTMLTableElement>(parent->parentNode());
     }
 
-    if (HTMLTableSectionElement* foot = downcast<HTMLTableElement>(*table).tFoot()) {
-        for (Node *row = foot->firstChild(); row; row = row->nextSibling()) {
-            if (row == this)
-                return rIndex;
-            if (row->hasTagName(trTag))
-                ++rIndex;
-        }
+    auto rows = table->rows();
+    unsigned length = rows->length();
+    for (unsigned i = 0; i < length; ++i) {
+        if (rows->item(i) == this)
+            return i;
     }
 
-    // We get here for rows that are in <thead> or <tfoot> sections other than the main header and footer.
     return -1;
 }
 
 int HTMLTableRowElement::sectionRowIndex() const
 {
-    int rIndex = 0;
-    const Node *n = this;
-    do {
-        n = n->previousSibling();
-        if (n && n->hasTagName(trTag))
-            rIndex++;
+    auto* parent = parentNode();
+    if (!parent)
+        return -1;
+
+    RefPtr<HTMLCollection> rows;
+    if (is<HTMLTableSectionElement>(*parent))
+        rows = downcast<HTMLTableSectionElement>(*parent).rows();
+    else if (is<HTMLTableElement>(*parent))
+        rows = downcast<HTMLTableElement>(*parent).rows();
+    else
+        return -1;
+
+    unsigned length = rows->length();
+    for (unsigned i = 0; i < length; ++i) {
+        if (rows->item(i) == this)
+            return i;
     }
-    while (n);
 
-    return rIndex;
+    return -1;
 }
 
-RefPtr<HTMLElement> HTMLTableRowElement::insertCell(int index, ExceptionCode& ec)
+RefPtr<HTMLTableCellElement> HTMLTableRowElement::insertCell(int index, ExceptionCode& ec)
 {
     Ref<HTMLCollection> children = cells();
     int numCells = children->length();
     if (index < -1 || index > numCells) {
         ec = INDEX_SIZE_ERR;
-        return 0;
+        return nullptr;
     }
 
-    RefPtr<HTMLTableCellElement> cell = HTMLTableCellElement::create(tdTag, document());
+    auto cell = HTMLTableDataCellElement::create(document());
     if (index < 0 || index >= numCells)
-        appendChild(cell, ec);
+        appendChild(cell.copyRef(), ec);
     else {
         Node* n;
         if (index < 1)
             n = firstChild();
         else
             n = children->item(index);
-        insertBefore(cell, n, ec);
+        insertBefore(cell.copyRef(), n, ec);
     }
-    return cell;
+    return WTFMove(cell);
 }
 
 void HTMLTableRowElement::deleteCell(int index, ExceptionCode& ec)
@@ -148,10 +132,9 @@ void HTMLTableRowElement::deleteCell(int index, ExceptionCode& ec)
     int numCells = children->length();
     if (index == -1)
         index = numCells-1;
-    if (index >= 0 && index < numCells) {
-        RefPtr<Node> cell = children->item(index);
-        HTMLElement::removeChild(cell.get(), ec);
-    } else
+    if (index >= 0 && index < numCells)
+        HTMLElement::removeChild(*children->item(index), ec);
+    else
         ec = INDEX_SIZE_ERR;
 }