Node::lazyAttach shouldn't lie about being attached

Source/core/html/HTMLPlugInImageElement.cpp

Index: Source/core/html/HTMLPlugInImageElement.cpp
diff --git a/Source/core/html/HTMLPlugInImageElement.cpp b/Source/core/html/HTMLPlugInImageElement.cpp
index b5512f38c906350c74bbff3e22a4261840de4ce2..441921e455a2ab0ee4561e4f39b968cf42eaa5c1 100644
--- a/Source/core/html/HTMLPlugInImageElement.cpp
+++ b/Source/core/html/HTMLPlugInImageElement.cpp
@@ -155,7 +155,9 @@ RenderObject* HTMLPlugInImageElement::createRenderer(RenderArena* arena, RenderS
 
 void HTMLPlugInImageElement::willRecalcStyle(StyleChange)
 {
-    // FIXME: Why is this necessary?  Manual re-attach is almost always wrong.
+    // FIXME: setNeedsWidgetUpdate is used by subclasses to signal that they need
+    // their renderer replaced on the next style recalc. It's not clear why they
+    // don't just do lazyReattach instead. This code looks like a terrible hack.
     if (!useFallbackContent() && needsWidgetUpdate() && renderer() && !isImageType())
         reattach();
 }