Node::lazyAttach shouldn't lie about being attached

Source/core/html/HTMLDialogElement.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 81 matching lines @@
     // FIXME: Figure out what to do in vertical writing mode.
     FrameView* frameView = document()->view();
     int scrollTop = frameView->scrollOffset().height();
     int visibleHeight = frameView->visibleContentRect(ScrollableArea::IncludeScrollbars).height();
     m_top = scrollTop;
     if (box->height() < visibleHeight)
         m_top += (visibleHeight - box->height()) / 2;
     m_topIsValid = true;
 
     // FIXME: It's inefficient to reattach here. We could do better by mutating style directly and forcing another layout.
-    reattach();
+    lazyReattach();
 }
 
 void HTMLDialogElement::show()
 {
     if (fastHasAttribute(openAttr))
         return;
     setBooleanAttribute(openAttr, true);
     positionAndReattach();
 }
 
@@ skipping 19 matching lines @@
 }
 
 bool HTMLDialogElement::shouldBeReparentedUnderRenderView(const RenderStyle* style) const
 {
     if (style && style->position() == AbsolutePosition)
         return true;
     return Element::shouldBeReparentedUnderRenderView(style);
 }
 
 } // namespace WebCore