Cleanup of blink::HTMLFieldSetElement

third_party/WebKit/Source/core/html/HTMLFieldSetElement.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, 2010 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
  * License as published by the Free Software Foundation; either
@@ skipping 10 matching lines @@
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #include "core/html/HTMLFieldSetElement.h"
 
 #include "core/HTMLNames.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/NodeListsNodeData.h"
 #include "core/html/HTMLCollection.h"
-#include "core/html/HTMLFormControlsCollection.h"
 #include "core/html/HTMLLegendElement.h"
-#include "core/html/HTMLObjectElement.h"
 #include "core/layout/LayoutFieldset.h"
 #include "wtf/StdLibExtras.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 inline HTMLFieldSetElement::HTMLFieldSetElement(Document& document,
                                                 HTMLFormElement* form)
-    : HTMLFormControlElement(fieldsetTag, document, form),
-      m_documentVersion(0) {}
+    : HTMLFormControlElement(fieldsetTag, document, form) {}
 
 HTMLFieldSetElement* HTMLFieldSetElement::create(Document& document,
                                                  HTMLFormElement* form) {
   return new HTMLFieldSetElement(document, form);
 }
 
-DEFINE_TRACE(HTMLFieldSetElement) {
-  visitor->trace(m_listedElements);
-  HTMLFormControlElement::trace(visitor);
-}
-
 bool HTMLFieldSetElement::matchesValidityPseudoClasses() const {
   return true;
 }
 
 bool HTMLFieldSetElement::isValidElement() {
-  const ListedElement::List& elements = listedElements();
-  for (unsigned i = 0; i < elements.size(); ++i) {
-    if (elements[i]->isFormControlElement()) {
-      HTMLFormControlElement* control =
-          toHTMLFormControlElement(elements[i].get());
-      if (!control->checkValidity(0, CheckValidityDispatchNoEvent))
+  HTMLCollection* elements = this->elements();
+  for (unsigned i = 0; i < elements->length(); ++i) {
+    if (elements->item(i)->isFormControlElement()) {
+      if (!toHTMLFormControlElement(elements->item(i))
+               ->checkValidity(nullptr, CheckValidityDispatchNoEvent))
         return false;
     }
   }
   return true;
 }
 
 bool HTMLFieldSetElement::isSubmittableElement() {
   return false;
 }
 
@@ skipping 31 matching lines @@
 }
 
 HTMLLegendElement* HTMLFieldSetElement::legend() const {
   return Traversal<HTMLLegendElement>::firstChild(*this);
 }
 
 HTMLCollection* HTMLFieldSetElement::elements() {
   return ensureCachedCollection<HTMLCollection>(FormControls);
 }
 
-void HTMLFieldSetElement::refreshElementsIfNeeded() const {
-  uint64_t docVersion = document().domTreeVersion();
-  if (m_documentVersion == docVersion)
-    return;
-
-  m_documentVersion = docVersion;
-
-  m_listedElements.clear();
-
-  for (HTMLElement& element : Traversal<HTMLElement>::descendantsOf(*this)) {
-    if (isHTMLObjectElement(element)) {
-      m_listedElements.append(toHTMLObjectElement(&element));
-      continue;
-    }
-
-    if (!element.isFormControlElement())
-      continue;
-
-    m_listedElements.append(toHTMLFormControlElement(&element));
-  }
-}
-
-const ListedElement::List& HTMLFieldSetElement::listedElements() const {
-  refreshElementsIfNeeded();
-  return m_listedElements;
-}
-
 int HTMLFieldSetElement::tabIndex() const {
   return HTMLElement::tabIndex();
 }
 
 }  // namespace blink