SELECT element: Simplify the code by optionList().

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

 /*
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  * 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, 2009, 2010, 2011 Apple Inc. All rights reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
@@ skipping 262 matching lines @@
 
 String HTMLSelectElement::value() const
 {
     if (HTMLOptionElement* option = selectedOption())
         return option->value();
     return "";
 }
 
 void HTMLSelectElement::setValue(const String &value, bool sendEvents)
 {
-    // We clear the previously selected option(s) when needed, to guarantee
-    // calling setSelectedIndex() only once.
-    int optionIndex = 0;
     HTMLOptionElement* option = nullptr;
-    if (value.isNull()) {
-        optionIndex = -1;
-    } else {
-        // Find the option with value() matching the given parameter and make it
-        // the current selection.
-        for (auto& item : listItems()) {
-            if (!isHTMLOptionElement(item))
-                continue;
-            if (toHTMLOptionElement(item)->value() == value) {
-                option = toHTMLOptionElement(item);
-                break;
-            }
-            optionIndex++;
+    // Find the option with value() matching the given parameter and make it the
+    // current selection.
+    for (const auto& item : optionList()) {
+        if (item->value() == value) {
+            option = item;
+            break;
         }
-        if (optionIndex >= static_cast<int>(listItems().size()))
-            optionIndex = -1;
     }
 
-    int previousSelectedIndex = selectedIndex();
+    HTMLOptionElement* previousSelectedOption = selectedOption();
     setSuggestedOption(nullptr);
     if (m_isAutofilledByPreview)
         setAutofilled(false);
     SelectOptionFlags flags = DeselectOtherOptions | MakeOptionDirty;
     if (sendEvents)
         flags |= DispatchInputAndChangeEvent;
     selectOption(option, flags);
 
-    if (sendEvents && previousSelectedIndex != selectedIndex() && !usesMenuList())
+    if (sendEvents && previousSelectedOption != option && !usesMenuList())
         listBoxOnChange();
 }
 
 String HTMLSelectElement::suggestedValue() const
 {
     return m_suggestedOption ? m_suggestedOption->value() : "";
 }
 
 void HTMLSelectElement::setSuggestedValue(const String& value)
 {
     if (value.isNull()) {
         setSuggestedOption(nullptr);
         return;
     }
 
-    for (auto& item : listItems()) {
-        if (!isHTMLOptionElement(item))
-            continue;
-        if (toHTMLOptionElement(item)->value() == value) {
-            setSuggestedOption(toHTMLOptionElement(item));
+    for (const auto& option : optionList()) {
+        if (option->value() == value) {
+            setSuggestedOption(option);
             m_isAutofilledByPreview = true;
             return;
         }
     }
 
     setSuggestedOption(nullptr);
 }
 
 bool HTMLSelectElement::isPresentationAttribute(const QualifiedName& name) const
 {
@@ skipping 145 matching lines @@
     if (diff < 0) { // Add dummy elements.
         do {
             appendChild(document().createElement(optionTag, CreatedByCreateElement), exceptionState);
             if (exceptionState.hadException())
                 break;
         } while (++diff);
     } else {
         // Removing children fires mutation events, which might mutate the DOM
         // further, so we first copy out a list of elements that we intend to
         // remove then attempt to remove them one at a time.
-        HeapVector<Member<Element>> itemsToRemove;
+        HeapVector<Member<HTMLOptionElement>> itemsToRemove;
         size_t optionIndex = 0;
-        for (auto& item : listItems()) {
-            if (isHTMLOptionElement(item) && optionIndex++ >= newLen) {
-                ASSERT(item->parentNode());
-                itemsToRemove.append(item.get());
+        for (const auto& option : optionList()) {
+            if (optionIndex++ >= newLen) {
+                DCHECK(option->parentNode());
+                itemsToRemove.append(option);
             }
         }
 
         for (auto& item : itemsToRemove) {
             if (item->parentNode())
                 item->parentNode()->removeChild(item.get(), exceptionState);
         }
     }
     setNeedsValidityCheck();
 }
@@ skipping 365 matching lines @@
             return option;
     }
     return nullptr;
 }
 
 int HTMLSelectElement::selectedIndex() const
 {
     unsigned index = 0;
 
     // Return the number of the first option selected.
-    for (auto& element : listItems()) {
-        if (!isHTMLOptionElement(*element))
-            continue;
-        if (toHTMLOptionElement(*element).selected())
+    for (const auto& option : optionList()) {
+        if (option->selected())
             return index;
         ++index;
     }
 
     return -1;
 }
 
 void HTMLSelectElement::setSelectedIndex(int index)
 {
     selectOption(item(index), DeselectOtherOptions | MakeOptionDirty);
@@ skipping 345 matching lines @@
             resetToDefaultSelection();
     }
 }
 
 void HTMLSelectElement::appendToFormData(FormData& formData)
 {
     const AtomicString& name = this->name();
     if (name.isEmpty())
         return;
 
-    for (auto& element : listItems()) {
-        if (isHTMLOptionElement(*element) && toHTMLOptionElement(*element).selected() && !toHTMLOptionElement(*element).isDisabledFormControl())
-            formData.append(name, toHTMLOptionElement(*element).value());
+    for (const auto& option : optionList()) {
+        if (option->selected() && !option->isDisabledFormControl())
+            formData.append(name, option->value());
     }
 }
 
 void HTMLSelectElement::resetImpl()
 {
-    for (auto& item : listItems()) {
-        if (!isHTMLOptionElement(item))
-            continue;
-        HTMLOptionElement* option = toHTMLOptionElement(item);
+    for (const auto& option : optionList()) {
         option->setSelectedState(option->fastHasAttribute(selectedAttr));
         option->setDirty(false);
     }
     resetToDefaultSelection();
     setNeedsValidityCheck();
 }
 
 void HTMLSelectElement::handlePopupOpenKeyboardEvent(Event* event)
 {
     focus();
@@ skipping 498 matching lines @@
     option->setDirty(true);
     if (usesMenuList())
         return;
     listBoxOnChange();
     scrollToSelection();
 }
 
 unsigned HTMLSelectElement::length() const
 {
     unsigned options = 0;
-    for (auto& item : listItems()) {
-        if (isHTMLOptionElement(*item))
-            ++options;
+    for (const auto& option : optionList()) {
+        ALLOW_UNUSED_LOCAL(option);
+        ++options;
     }
     return options;
 }
 
 void HTMLSelectElement::finishParsingChildren()
 {
     HTMLFormControlElementWithState::finishParsingChildren();
     if (usesMenuList())
         return;
     scrollToOption(selectedOption());
@@ skipping 285 matching lines @@
 }
 
 void HTMLSelectElement::didMutateSubtree()
 {
     DCHECK(popupIsVisible());
     DCHECK(m_popup);
     m_popup->updateFromElement(PopupMenu::ByDOMChange);
 }
 
 } // namespace blink