Introduce Element::AttributeModificationParams

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Stefan Schimanski (1Stein@gmx.de)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 75 matching lines @@
 void HTMLObjectElement::collectStyleForPresentationAttribute(
     const QualifiedName& name,
     const AtomicString& value,
     MutableStylePropertySet* style) {
   if (name == borderAttr)
     applyBorderAttributeToStyle(value, style);
   else
     HTMLPlugInElement::collectStyleForPresentationAttribute(name, value, style);
 }
 
-void HTMLObjectElement::parseAttribute(const QualifiedName& name,
-                                       const AtomicString& oldValue,
-                                       const AtomicString& value) {
+void HTMLObjectElement::parseAttribute(
+    const AttributeModificationParams& params) {
+  const QualifiedName& name = params.name;
   if (name == formAttr) {
     formAttributeChanged();
   } else if (name == typeAttr) {
-    m_serviceType = value.lower();
+    m_serviceType = params.newValue.lower();
     size_t pos = m_serviceType.find(";");
     if (pos != kNotFound)
       m_serviceType = m_serviceType.left(pos);
     // TODO(schenney): crbug.com/572908 What is the right thing to do here?
     // Should we suppress the reload stuff when a persistable widget-type is
     // specified?
     reloadPluginOnAttributeChange(name);
     if (!layoutObject())
       requestPluginCreationWithoutLayoutObjectIfPossible();
   } else if (name == dataAttr) {
-    m_url = stripLeadingAndTrailingHTMLSpaces(value);
+    m_url = stripLeadingAndTrailingHTMLSpaces(params.newValue);
     if (layoutObject() && isImageType()) {
       setNeedsWidgetUpdate(true);
       if (!m_imageLoader)
         m_imageLoader = HTMLImageLoader::create(this);
       m_imageLoader->updateFromElement(ImageLoader::UpdateIgnorePreviousError);
     } else {
       reloadPluginOnAttributeChange(name);
     }
   } else if (name == classidAttr) {
-    m_classId = value;
+    m_classId = params.newValue;
     reloadPluginOnAttributeChange(name);
   } else {
-    HTMLPlugInElement::parseAttribute(name, oldValue, value);
+    HTMLPlugInElement::parseAttribute(params);
   }
 }
 
 static void mapDataParamToSrc(Vector<String>* paramNames,
                               Vector<String>* paramValues) {
   // Some plugins don't understand the "data" attribute of the OBJECT tag (i.e.
   // Real and WMP require "src" attribute).
   int srcIndex = -1, dataIndex = -1;
   for (unsigned i = 0; i < paramNames->size(); ++i) {
     if (equalIgnoringCase((*paramNames)[i], "src"))
@@ skipping 321 matching lines @@
 
 bool HTMLObjectElement::willUseFallbackContentAtLayout() const {
   return !hasValidClassId() && hasFallbackContent();
 }
 
 void HTMLObjectElement::associateWith(HTMLFormElement* form) {
   associateByParser(form);
 };
 
 }  // namespace blink