HTMLTableCellElement type changed from signed to unsinged

Source/core/html/HTMLTableCellElement.cpp

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2010 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 26 matching lines @@
 using std::max;
 using std::min;
 
 namespace blink {
 
 // Clamp rowspan and colspan at 8k.
 // Firefox used a limit of 8190 for rowspan but they changed it to 65,534.
 // (FIXME: We should consider increasing this limit (crbug.com/78577).
 // Firefox uses a limit of 1,000 for colspan and resets the value to 1
 // but we don't discriminate between rowspan / colspan as it is artificial.
-static const int maxColRowSpan = 8190;
+static const unsigned maxColRowSpan = 8190;
 
 using namespace HTMLNames;
 
 inline HTMLTableCellElement::HTMLTableCellElement(const QualifiedName& tagName, Document& document)
     : HTMLTablePartElement(tagName, document)
 {
 }
 
 DEFINE_ELEMENT_FACTORY_WITH_TAGNAME(HTMLTableCellElement)
 
-int HTMLTableCellElement::colSpan() const
+unsigned HTMLTableCellElement::colSpan() const
 {
     const AtomicString& colSpanValue = fastGetAttribute(colspanAttr);
-    int value = 0;
-    if (colSpanValue.isEmpty() || !parseHTMLInteger(colSpanValue, value))
+    unsigned value = 0;
+    if (colSpanValue.isEmpty() || !parseHTMLNonNegativeInteger(colSpanValue, value))
         return 1;
-    return max(1, min(value, maxColRowSpan));
+    return max(1u, min(value, maxColRowSpan));
 }
 
-int HTMLTableCellElement::rowSpan() const
+unsigned HTMLTableCellElement::rowSpan() const
 {
     const AtomicString& rowSpanValue = fastGetAttribute(rowspanAttr);
-    int value = 0;
-    if (rowSpanValue.isEmpty() || !parseHTMLInteger(rowSpanValue, value))
+    unsigned value = 0;
+    if (rowSpanValue.isEmpty() || !parseHTMLNonNegativeInteger(rowSpanValue, value))
         return 1;
-    return max(1, min(value, maxColRowSpan));
+    return max(1u, min(value, maxColRowSpan));
 }
 
 int HTMLTableCellElement::cellIndex() const
 {
     if (!isHTMLTableRowElement(parentElement()))
         return -1;
 
     int index = 0;
     for (const HTMLTableCellElement* element = Traversal<HTMLTableCellElement>::previousSibling(*this); element; element = Traversal<HTMLTableCellElement>::previousSibling(*element))
         ++index;
@@ skipping 65 matching lines @@
 const AtomicString& HTMLTableCellElement::abbr() const
 {
     return fastGetAttribute(abbrAttr);
 }
 
 const AtomicString& HTMLTableCellElement::axis() const
 {
     return fastGetAttribute(axisAttr);
 }
 
-void HTMLTableCellElement::setColSpan(int n)
+void HTMLTableCellElement::setColSpan(unsigned n)
 {
-    setIntegralAttribute(colspanAttr, n);
+    setUnsignedIntegralAttribute(colspanAttr, n);
 }
 
 const AtomicString& HTMLTableCellElement::headers() const
 {
     return fastGetAttribute(headersAttr);
 }
 
-void HTMLTableCellElement::setRowSpan(int n)
+void HTMLTableCellElement::setRowSpan(unsigned n)
 {
-    setIntegralAttribute(rowspanAttr, n);
+    setUnsignedIntegralAttribute(rowspanAttr, n);
 }
 
 const AtomicString& HTMLTableCellElement::scope() const
 {
     return fastGetAttribute(scopeAttr);
 }
 
 HTMLTableCellElement* HTMLTableCellElement::cellAbove() const
 {
     LayoutObject* cellLayoutObject = layoutObject();
     if (!cellLayoutObject)
         return nullptr;
     if (!cellLayoutObject->isTableCell())
         return nullptr;
 
     LayoutTableCell* tableCellLayoutObject = toLayoutTableCell(cellLayoutObject);
     LayoutTableCell* cellAboveLayoutObject = tableCellLayoutObject->table()->cellAbove(tableCellLayoutObject);
     if (!cellAboveLayoutObject)
         return nullptr;
 
     return toHTMLTableCellElement(cellAboveLayoutObject->node());
 }
 
 } // namespace blink