Introduce Element::AttributeModificationParams

third_party/WebKit/Source/core/html/HTMLTableElement.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, 2008, 2010, 2011 Apple Inc. All rights
  * reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 368 matching lines @@
     const QualifiedName& name) const {
   if (name == widthAttr || name == heightAttr || name == bgcolorAttr ||
       name == backgroundAttr || name == valignAttr || name == vspaceAttr ||
       name == hspaceAttr || name == alignAttr || name == cellspacingAttr ||
       name == borderAttr || name == bordercolorAttr || name == frameAttr ||
       name == rulesAttr)
     return true;
   return HTMLElement::isPresentationAttribute(name);
 }
 
-void HTMLTableElement::parseAttribute(const QualifiedName& name,
-                                      const AtomicString& oldValue,
-                                      const AtomicString& value) {
+void HTMLTableElement::parseAttribute(
+    const AttributeModificationParams& params) {
+  const QualifiedName& name = params.name;
   CellBorders bordersBefore = getCellBorders();
   unsigned short oldPadding = m_padding;
 
   if (name == borderAttr) {
     // FIXME: This attribute is a mess.
-    m_borderAttr = parseBorderWidthAttribute(value);
+    m_borderAttr = parseBorderWidthAttribute(params.newValue);
   } else if (name == bordercolorAttr) {
-    m_borderColorAttr = !value.isEmpty();
+    m_borderColorAttr = !params.newValue.isEmpty();
   } else if (name == frameAttr) {
     // FIXME: This attribute is a mess.
     bool borderTop;
     bool borderRight;
     bool borderBottom;
     bool borderLeft;
     m_frameAttr = getBordersFromFrameAttributeValue(
-        value, borderTop, borderRight, borderBottom, borderLeft);
+        params.newValue, borderTop, borderRight, borderBottom, borderLeft);
   } else if (name == rulesAttr) {
     m_rulesAttr = UnsetRules;
-    if (equalIgnoringCase(value, "none"))
+    if (equalIgnoringCase(params.newValue, "none"))
       m_rulesAttr = NoneRules;
-    else if (equalIgnoringCase(value, "groups"))
+    else if (equalIgnoringCase(params.newValue, "groups"))
       m_rulesAttr = GroupsRules;
-    else if (equalIgnoringCase(value, "rows"))
+    else if (equalIgnoringCase(params.newValue, "rows"))
       m_rulesAttr = RowsRules;
-    else if (equalIgnoringCase(value, "cols"))
+    else if (equalIgnoringCase(params.newValue, "cols"))
       m_rulesAttr = ColsRules;
-    else if (equalIgnoringCase(value, "all"))
+    else if (equalIgnoringCase(params.newValue, "all"))
       m_rulesAttr = AllRules;
-  } else if (name == cellpaddingAttr) {
-    if (!value.isEmpty())
-      m_padding = std::max(0, value.toInt());
+  } else if (params.name == cellpaddingAttr) {
+    if (!params.newValue.isEmpty())
+      m_padding = std::max(0, params.newValue.toInt());
     else
       m_padding = 1;
-  } else if (name == colsAttr) {
+  } else if (params.name == colsAttr) {
     // ###
   } else {
-    HTMLElement::parseAttribute(name, oldValue, value);
+    HTMLElement::parseAttribute(params);
   }
 
   if (bordersBefore != getCellBorders() || oldPadding != m_padding) {
     m_sharedCellStyle = nullptr;
     setNeedsTableStyleRecalc();
   }
 }
 
 static StylePropertySet* createBorderStyle(CSSValueID value) {
   MutableStylePropertySet* style =
@@ skipping 167 matching lines @@
 const AtomicString& HTMLTableElement::summary() const {
   return getAttribute(summaryAttr);
 }
 
 DEFINE_TRACE(HTMLTableElement) {
   visitor->trace(m_sharedCellStyle);
   HTMLElement::trace(visitor);
 }
 
 }  // namespace blink