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 fadee57c133dedc721f148ef16f8c8f4af119325..1a781ab71cd56440255856d47639f8979104c0be 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();
 }