Rename Node::attached() to confusingAndOftenMisusedAttached()

Source/core/page/Frame.cpp

Index: Source/core/page/Frame.cpp
diff --git a/Source/core/page/Frame.cpp b/Source/core/page/Frame.cpp
index 7e1e409b33df21623d2aadbf9e3ee0d334fc5c01..4e5dd45814f751cfe5b78e34a0fb6ef78289ac4a 100644
--- a/Source/core/page/Frame.cpp
+++ b/Source/core/page/Frame.cpp
@@ -190,7 +190,7 @@ void Frame::setView(PassRefPtr<FrameView> view)
     // Prepare for destruction now, so any unload event handlers get run and the DOMWindow is
     // notified. If we wait until the view is destroyed, then things won't be hooked up enough for
     // these calls to work.
-    if (!view && document() && document()->attached()) {
+    if (!view && document() && document()->confusingAndOftenMisusedAttached()) {
         // FIXME: We don't call willRemove here. Why is that OK?
         document()->prepareForDestruction();
     }