Improve performance to remove OPTIONs from a single-selection SELECT.

third_party/WebKit/Source/core/layout/LayoutMenuList.cpp

 /*
  * This file is part of the select element layoutObject in WebCore.
  *
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  *               2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.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 141 matching lines @@
     LayoutBlock::styleDidChange(diff, oldStyle);
 
     if (!m_innerBlock)
         createInnerBlock();
 
     m_buttonText->setStyle(mutableStyle());
     adjustInnerStyle();
 
     bool fontChanged = !oldStyle || oldStyle->font() != style()->font();
     if (fontChanged)
-        updateOptionsWidth();
+        m_optionsChanged = true;
 }
 
-void LayoutMenuList::updateOptionsWidth()
+void LayoutMenuList::updateOptionsWidth() const
 {
+    if (!m_optionsChanged)
+        return;
+
     float maxOptionWidth = 0;
     const HeapVector<Member<HTMLElement>>& listItems = selectElement()->listItems();
     int size = listItems.size();
 
     for (int i = 0; i < size; ++i) {
         HTMLElement* element = listItems[i];
         if (!isHTMLOptionElement(*element))
             continue;
 
         String text = toHTMLOptionElement(element)->textIndentedToRespectGroupLabel();
         applyTextTransform(style(), text, ' ');
         if (LayoutTheme::theme().popupOptionSupportsTextIndent()) {
             // Add in the option's text indent.  We can't calculate percentage values for now.
             float optionWidth = 0;
             if (const ComputedStyle* optionStyle = element->computedStyle())
                 optionWidth += minimumValueForLength(optionStyle->textIndent(), LayoutUnit());
             if (!text.isEmpty())
                 optionWidth += computeTextWidth(text);
             maxOptionWidth = std::max(maxOptionWidth, optionWidth);
         } else if (!text.isEmpty()) {
             maxOptionWidth = std::max(maxOptionWidth, computeTextWidth(text));
         }
     }
 
-    int width = static_cast<int>(ceilf(maxOptionWidth));
-    if (m_optionsWidth == width)
-        return;
+    m_optionsWidth = static_cast<int>(ceilf(maxOptionWidth));
+    m_optionsChanged = false;
+}
 
-    m_optionsWidth = width;
-    if (parent())
-        setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::MenuWidthChanged);
+void LayoutMenuList::setOptionsChanged(bool changed)
+{
+    m_optionsChanged = changed;
+    if (changed && parent())
+        setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::MenuOptionsChanged);
 }
 
 float LayoutMenuList::computeTextWidth(const String& text) const
 {
     return style()->font().width(constructTextRun(style()->font(), text, styleRef()));
 }
 
 void LayoutMenuList::updateFromElement()
 {
-    if (m_optionsChanged) {
-        updateOptionsWidth();
-        m_optionsChanged = false;
-    }
-
-    updateText();
-}
-
-void LayoutMenuList::updateText()
-{
     setTextFromOption(selectElement()->optionIndexToBeShown());
 }
 
 void LayoutMenuList::setTextFromOption(int optionIndex)
 {
     HTMLSelectElement* select = selectElement();
     const HeapVector<Member<HTMLElement>>& listItems = select->listItems();
     const int size = listItems.size();
 
     String text = emptyString();
@@ skipping 74 matching lines @@
 
     LayoutRect innerBox(additionalOffset + m_innerBlock->location()
         + LayoutSize(m_innerBlock->paddingLeft(), m_innerBlock->paddingTop())
         , m_innerBlock->contentSize());
 
     return intersection(outerBox, innerBox);
 }
 
 void LayoutMenuList::computeIntrinsicLogicalWidths(LayoutUnit& minLogicalWidth, LayoutUnit& maxLogicalWidth) const
 {
+    updateOptionsWidth();
     maxLogicalWidth = std::max(m_optionsWidth, LayoutTheme::theme().minimumMenuListSize(styleRef())) + m_innerBlock->paddingLeft() + m_innerBlock->paddingRight();
     if (!style()->width().hasPercent())
         minLogicalWidth = maxLogicalWidth;
 }
 
 void LayoutMenuList::didSetSelectedIndex(int optionIndex)
 {
     didUpdateActiveOption(optionIndex);
 }
 
@@ skipping 36 matching lines @@
         // space appear on the side of the popup menu.
         return LayoutUnit(endOfLinePadding);
     }
 
     // If the appearance isn't MenulistPart, then the select is styled (non-native), so
     // we want to return the user specified padding.
     return paddingRight() + m_innerBlock->paddingRight();
 }
 
 } // namespace blink