Revert 150924 "Node::lazyAttach shouldn't lie about being attached"

trunk/Source/core/html/HTMLPlugInImageElement.cpp

Index: trunk/Source/core/html/HTMLPlugInImageElement.cpp
===================================================================
--- trunk/Source/core/html/HTMLPlugInImageElement.cpp	(revision 151673)
+++ trunk/Source/core/html/HTMLPlugInImageElement.cpp	(working copy)
@@ -154,9 +154,7 @@
 
 void HTMLPlugInImageElement::willRecalcStyle(StyleChange)
 {
-    // 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.
+    // FIXME: Why is this necessary?  Manual re-attach is almost always wrong.
     if (!useFallbackContent() && needsWidgetUpdate() && renderer() && !isImageType())
         reattach();
 }