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 84 matching lines @@
     FloatPoint absolutePoint(0, scrollTop);
     int visibleHeight = frameView->visibleContentRect(ScrollableArea::IncludeScrollbars).height();
     if (box->height() < visibleHeight)
         absolutePoint.move(0, (visibleHeight - box->height()) / 2);
     FloatPoint localPoint = box->containingBlock()->absoluteToLocal(absolutePoint);
 
     m_top = localPoint.y();
     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 12 matching lines @@
 {
     // FIXME: Workaround for <https://bugs.webkit.org/show_bug.cgi?id=91058>: modifying an attribute for which there is an attribute selector
     // in html.css sometimes does not trigger a style recalc.
     if (name == openAttr)
         return true;
 
     return HTMLElement::isPresentationAttribute(name);
 }
 
 } // namespace WebCore