Introduce Element::AttributeModificationParams

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights
  * reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 458 matching lines @@
   // structure.
   ListedElement::List elements(listedElements());
   for (const auto& element : elements) {
     if (element->isFormControlElement())
       toHTMLFormControlElement(element)->reset();
   }
 
   m_isInResetFunction = false;
 }
 
-void HTMLFormElement::parseAttribute(const QualifiedName& name,
-                                     const AtomicString& oldValue,
-                                     const AtomicString& value) {
+void HTMLFormElement::parseAttribute(
+    const AttributeModificationParams& params) {
+  const QualifiedName& name = params.name;
   if (name == actionAttr) {
-    m_attributes.parseAction(value);
-    logUpdateAttributeIfIsolatedWorldAndInDocument("form", actionAttr, oldValue,
-                                                   value);
+    m_attributes.parseAction(params.newValue);
+    logUpdateAttributeIfIsolatedWorldAndInDocument("form", params);
 
     // If we're not upgrading insecure requests, and the new action attribute is
     // pointing to an insecure "action" location from a secure page it is marked
     // as "passive" mixed content.
     if (document().getInsecureRequestPolicy() & kUpgradeInsecureRequests)
       return;
     KURL actionURL = document().completeURL(m_attributes.action().isEmpty()
                                                 ? document().url().getString()
                                                 : m_attributes.action());
     if (MixedContentChecker::isMixedFormAction(document().frame(), actionURL))
       UseCounter::count(document().frame(),
                         UseCounter::MixedContentFormPresent);
   } else if (name == targetAttr) {
-    m_attributes.setTarget(value);
+    m_attributes.setTarget(params.newValue);
   } else if (name == methodAttr) {
-    m_attributes.updateMethodType(value);
+    m_attributes.updateMethodType(params.newValue);
   } else if (name == enctypeAttr) {
-    m_attributes.updateEncodingType(value);
+    m_attributes.updateEncodingType(params.newValue);
   } else if (name == accept_charsetAttr) {
-    m_attributes.setAcceptCharset(value);
+    m_attributes.setAcceptCharset(params.newValue);
   } else {
-    HTMLElement::parseAttribute(name, oldValue, value);
+    HTMLElement::parseAttribute(params);
   }
 }
 
 void HTMLFormElement::associate(ListedElement& e) {
   m_listedElementsAreDirty = true;
   m_listedElements.clear();
   if (toHTMLElement(e).fastHasAttribute(formAttr))
     m_hasElementsAssociatedByFormAttribute = true;
 }
 
@@ skipping 289 matching lines @@
   for (const auto& control : listedElements()) {
     if (!control->isFormControlElement())
       continue;
     if (toHTMLFormControlElement(control)->canBeSuccessfulSubmitButton())
       toHTMLFormControlElement(control)->pseudoStateChanged(
           CSSSelector::PseudoDefault);
   }
 }
 
 }  // namespace blink