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 5767d025b444c05687f572eaa833dbe5b6765db0..dee122ad1d482b6621f895f33ee43538469d98b8 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();
         }