Introduce Element::AttributeModificationParams

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Simon Hausmann (hausmann@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2006, 2009, 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
  * License as published by the Free Software Foundation; either
@@ skipping 52 matching lines @@
 void HTMLFrameSetElement::collectStyleForPresentationAttribute(
     const QualifiedName& name,
     const AtomicString& value,
     MutableStylePropertySet* style) {
   if (name == bordercolorAttr)
     addHTMLColorToStyle(style, CSSPropertyBorderColor, value);
   else
     HTMLElement::collectStyleForPresentationAttribute(name, value, style);
 }
 
-void HTMLFrameSetElement::parseAttribute(const QualifiedName& name,
-                                         const AtomicString& oldValue,
-                                         const AtomicString& value) {
+void HTMLFrameSetElement::parseAttribute(
+    const AttributeModificationParams& params) {
+  const QualifiedName& name = params.name;
+  const AtomicString& value = params.newValue;
   if (name == rowsAttr) {
     if (!value.isNull()) {
       m_rowLengths = parseListOfDimensions(value.getString());
       setNeedsStyleRecalc(SubtreeStyleChange,
                           StyleChangeReasonForTracing::fromAttribute(name));
     }
   } else if (name == colsAttr) {
     if (!value.isNull()) {
       m_colLengths = parseListOfDimensions(value.getString());
       setNeedsStyleRecalc(SubtreeStyleChange,
@@ skipping 118 matching lines @@
     document().setWindowAttributeEventListener(
         EventTypeNames::popstate,
         createAttributeEventListener(document().frame(), name, value,
                                      eventParameterName()));
   } else if (name == onlanguagechangeAttr) {
     document().setWindowAttributeEventListener(
         EventTypeNames::languagechange,
         createAttributeEventListener(document().frame(), name, value,
                                      eventParameterName()));
   } else {
-    HTMLElement::parseAttribute(name, oldValue, value);
+    HTMLElement::parseAttribute(params);
   }
 }
 
 bool HTMLFrameSetElement::layoutObjectIsNeeded(const ComputedStyle& style) {
   // For compatibility, frames layoutObject even when display: none is set.
   // However, we delay creating a layoutObject until stylesheets have loaded.
   return style.isStyleAvailable();
 }
 
 LayoutObject* HTMLFrameSetElement::createLayoutObject(
@@ skipping 58 matching lines @@
   Element* frameElement = children()->namedItem(name);
   if (!isHTMLFrameElement(frameElement))
     return nullptr;
   Document* document = toHTMLFrameElement(frameElement)->contentDocument();
   if (!document || !document->frame())
     return nullptr;
   return document->domWindow();
 }
 
 }  // namespace blink