SELECT element: Avoid to use listItems() in HTMLSelectElement::selectOption()

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2004, 2005, 2006, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2011 Motorola Mobility, Inc.  All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 165 matching lines @@
 
 int HTMLOptionElement::index() const
 {
     // It would be faster to cache the index, but harder to get it right in all cases.
 
     HTMLSelectElement* selectElement = ownerSelectElement();
     if (!selectElement)
         return 0;
 
     int optionIndex = 0;
-
-    const HeapVector<Member<HTMLElement>>& items = selectElement->listItems();
-    size_t length = items.size();
-    for (size_t i = 0; i < length; ++i) {
-        if (!isHTMLOptionElement(*items[i]))
-            continue;
-        if (items[i].get() == this)
+    for (const auto& option : selectElement->optionList()) {
+        if (option == this)
             return optionIndex;
         ++optionIndex;
     }
 
     return 0;
 }
 
 int HTMLOptionElement::listIndex() const
 {
     if (HTMLSelectElement* selectElement = ownerSelectElement())
@@ skipping 269 matching lines @@
         ASSERT(parent);
         if (isHTMLOptGroupElement(*parent)) {
             const ComputedStyle* parentStyle = parent->computedStyle() ? parent->computedStyle() : parent->ensureComputedStyle();
             return !parentStyle || parentStyle->display() == NONE;
         }
     }
     return m_style->display() == NONE;
 }
 
 } // namespace blink