Node::lazyAttach shouldn't lie about being attached

Source/core/html/HTMLSelectElement.cpp

Index: Source/core/html/HTMLSelectElement.cpp
diff --git a/Source/core/html/HTMLSelectElement.cpp b/Source/core/html/HTMLSelectElement.cpp
index 01f61415518e9deebd89870d00a20b9b447ce096..6ce29fefb94516fda66069cd550b6343ea993109 100644
--- a/Source/core/html/HTMLSelectElement.cpp
+++ b/Source/core/html/HTMLSelectElement.cpp
@@ -307,6 +307,9 @@ void HTMLSelectElement::parseAttribute(const QualifiedName& name, const AtomicSt
         m_size = size;
         setNeedsValidityCheck();
         if (m_size != oldSize && attached()) {
+            // FIXME: setRecalcListItems uses the renderer to decide if we should update
+            // the accessibility tree so we can't lazyAttach. That code should be moved
+            // out of setRecalcListItems.
             reattach();
             setRecalcListItems();
         }