Delay DOM mutation events in HTMLSelectElement::setOption().

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 411 matching lines @@
     element.setHTMLOptionElement(option);
     HTMLElementOrLong before;
     // Out of array bounds? First insert empty dummies.
     if (diff > 0) {
         setLength(index, exceptionState);
         // Replace an existing entry?
     } else if (diff < 0) {
         before.setHTMLElement(options()->item(index + 1));
         remove(index);
     }
+    if (exceptionState.hadException())
+        return;
     // Finally add the new element.
-    if (!exceptionState.hadException()) {
-        add(element, before, exceptionState);
-        if (diff >= 0 && option->selected())
-            optionSelectionStateChanged(option, true);
-    }
+    EventQueueScope scope;
+    add(element, before, exceptionState);
+    if (diff >= 0 && option->selected())
+        optionSelectionStateChanged(option, true);
 }
 
 void HTMLSelectElement::setLength(unsigned newLen, ExceptionState& exceptionState)
 {
     // We should check |newLen > maxListItems| first to avoid integer overflow.
     if (newLen > maxListItems || listItems().size() + newLen - length() > maxListItems) {
         document().addConsoleMessage(ConsoleMessage::create(JSMessageSource, WarningMessageLevel,
             String::format("Blocked to expand the option list to %u items.  The maximum list length is %u.", newLen, maxListItems)));
         return;
     }
@@ skipping 1590 matching lines @@
 }
 
 void HTMLSelectElement::didMutateSubtree()
 {
     DCHECK(popupIsVisible());
     DCHECK(m_popup);
     m_popup->updateFromElement(PopupMenu::ByDOMChange);
 }
 
 } // namespace blink