Introduce Element::AttributeModificationParams

third_party/WebKit/Source/core/html/HTMLOptionElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2004, 2005, 2006, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2011 Motorola Mobility, Inc.  All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 177 matching lines @@
 
   return 0;
 }
 
 int HTMLOptionElement::listIndex() const {
   if (HTMLSelectElement* selectElement = ownerSelectElement())
     return selectElement->listIndexForOption(*this);
   return -1;
 }
 
-void HTMLOptionElement::parseAttribute(const QualifiedName& name,
-                                       const AtomicString& oldValue,
-                                       const AtomicString& value) {
+void HTMLOptionElement::parseAttribute(
+    const AttributeModificationParams& params) {
+  const QualifiedName& name = params.name;
   if (name == valueAttr) {
     if (HTMLDataListElement* dataList = ownerDataListElement())
       dataList->optionElementChildrenChanged();
   } else if (name == disabledAttr) {
-    if (oldValue.isNull() != value.isNull()) {
+    if (params.oldValue.isNull() != params.newValue.isNull()) {
       pseudoStateChanged(CSSSelector::PseudoDisabled);
       pseudoStateChanged(CSSSelector::PseudoEnabled);
       if (layoutObject())
         LayoutTheme::theme().controlStateChanged(*layoutObject(),
                                                  EnabledControlState);
     }
   } else if (name == selectedAttr) {
-    if (oldValue.isNull() != value.isNull() && !m_isDirty)
-      setSelected(!value.isNull());
+    if (params.oldValue.isNull() != params.newValue.isNull() && !m_isDirty)
+      setSelected(!params.newValue.isNull());
     pseudoStateChanged(CSSSelector::PseudoDefault);
   } else if (name == labelAttr) {
     updateLabel();
   } else {
-    HTMLElement::parseAttribute(name, oldValue, value);
+    HTMLElement::parseAttribute(params);
   }
 }
 
 String HTMLOptionElement::value() const {
   const AtomicString& value = fastGetAttribute(valueAttr);
   if (!value.isNull())
     return value;
   return collectOptionInnerText()
       .stripWhiteSpace(isHTMLSpace<UChar>)
       .simplifyWhiteSpace(isHTMLSpace<UChar>);
@@ skipping 224 matching lines @@
 }
 
 String HTMLOptionElement::innerText() {
   // A workaround for crbug.com/424578. We add ShadowRoot to an OPTION, but
   // innerText behavior for Shadow DOM is unclear.  We just return the same
   // string before adding ShadowRoot.
   return textContent();
 }
 
 }  // namespace blink